diff options
author | whunt@chromium.org <whunt@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-13 06:40:36 +0000 |
---|---|---|
committer | whunt@chromium.org <whunt@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-13 06:40:36 +0000 |
commit | a689854e1e36944e38857a07db3164fef5b4d978 (patch) | |
tree | 165a17b6e908201b44dfcb77b56fef5c313dd8d0 /ui/gfx | |
parent | c9bc287c1f517b95acbc8978e17e39089b0a09e6 (diff) | |
download | chromium_src-a689854e1e36944e38857a07db3164fef5b4d978.zip chromium_src-a689854e1e36944e38857a07db3164fef5b4d978.tar.gz chromium_src-a689854e1e36944e38857a07db3164fef5b4d978.tar.bz2 |
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
Diffstat (limited to 'ui/gfx')
-rw-r--r-- | ui/gfx/quad_f.cc | 7 | ||||
-rw-r--r-- | ui/gfx/quad_f.h | 1 |
2 files changed, 0 insertions, 8 deletions
diff --git a/ui/gfx/quad_f.cc b/ui/gfx/quad_f.cc index 8f2b0dc..aba67a3 100644 --- a/ui/gfx/quad_f.cc +++ b/ui/gfx/quad_f.cc @@ -11,13 +11,6 @@ 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) { p1_ = PointF(rect.x(), rect.y()); p2_ = PointF(rect.right(), rect.y()); diff --git a/ui/gfx/quad_f.h b/ui/gfx/quad_f.h index a63dcfc..faf2149 100644 --- a/ui/gfx/quad_f.h +++ b/ui/gfx/quad_f.h @@ -30,7 +30,6 @@ class UI_EXPORT QuadF { p3_(rect.right(), rect.bottom()), p4_(rect.x(), rect.bottom()) {} - void operator=(const QuadF& quad); void operator=(const RectF& rect); void set_p1(const PointF& p) { p1_ = p; } |