Commit a689854e authored by whunt@chromium.org's avatar whunt@chromium.org

Removed gfx::QuadF::operator =(const gfx::QuadF&) and let the compiler generate one

This function showed up as hot in the trace and should not have been explicitly implemented as non-inline (pushing the args for the function call costs as much as the copy itself so operator = should *never* not be inlined if it's just doing a normal copy)

BUG=175855


Review URL: https://chromiumcodereview.appspot.com/12250019

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182148 0039d316-1c4b-4281-b951-d872f2087c98
parent c9bc287c
...@@ -11,13 +11,6 @@ ...@@ -11,13 +11,6 @@
namespace gfx { namespace gfx {
void QuadF::operator=(const QuadF& quad) {
p1_ = quad.p1_;
p2_ = quad.p2_;
p3_ = quad.p3_;
p4_ = quad.p4_;
}
void QuadF::operator=(const RectF& rect) { void QuadF::operator=(const RectF& rect) {
p1_ = PointF(rect.x(), rect.y()); p1_ = PointF(rect.x(), rect.y());
p2_ = PointF(rect.right(), rect.y()); p2_ = PointF(rect.right(), rect.y());
......
...@@ -30,7 +30,6 @@ class UI_EXPORT QuadF { ...@@ -30,7 +30,6 @@ class UI_EXPORT QuadF {
p3_(rect.right(), rect.bottom()), p3_(rect.right(), rect.bottom()),
p4_(rect.x(), rect.bottom()) {} p4_(rect.x(), rect.bottom()) {}
void operator=(const QuadF& quad);
void operator=(const RectF& rect); void operator=(const RectF& rect);
void set_p1(const PointF& p) { p1_ = p; } void set_p1(const PointF& p) { p1_ = p; }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment