diff options
author | prashant.n@samsung.com <prashant.n@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-10 20:05:06 +0000 |
---|---|---|
committer | prashant.n@samsung.com <prashant.n@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-10 20:05:06 +0000 |
commit | 0023fc78d056556b353ee17f03760bec8c1fee49 (patch) | |
tree | 009367358a1c44b08f3142f63b087d936206ba44 /cc/test/render_pass_test_utils.cc | |
parent | 37df32bd517577ac6ad20c6d76b236265caeacaf (diff) | |
download | chromium_src-0023fc78d056556b353ee17f03760bec8c1fee49.zip chromium_src-0023fc78d056556b353ee17f03760bec8c1fee49.tar.gz chromium_src-0023fc78d056556b353ee17f03760bec8c1fee49.tar.bz2 |
Pass gfx::Rect and gfx::RectF by const ref.
Avoid unneccessary copy of structures gfx::Rect & gfx::RectF
by passing them by const ref rather than value.
Any struct of size > 4 bytes should be passed by const ref.
Passing by ref for these structs is faster than passing
by value, especially when invoking function has multiple parameters.
Pass by value creates unneccessary overhead which should be avoided.
BUG=159273
Review URL: https://codereview.chromium.org/93663004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@244224 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/test/render_pass_test_utils.cc')
-rw-r--r-- | cc/test/render_pass_test_utils.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/cc/test/render_pass_test_utils.cc b/cc/test/render_pass_test_utils.cc index 32e6f48..8b018cf 100644 --- a/cc/test/render_pass_test_utils.cc +++ b/cc/test/render_pass_test_utils.cc @@ -20,7 +20,7 @@ namespace cc { TestRenderPass* AddRenderPass(RenderPassList* pass_list, RenderPass::Id id, - gfx::Rect output_rect, + const gfx::Rect& output_rect, const gfx::Transform& root_transform) { scoped_ptr<TestRenderPass> pass(TestRenderPass::Create()); pass->SetNew(id, output_rect, output_rect, root_transform); @@ -30,7 +30,7 @@ TestRenderPass* AddRenderPass(RenderPassList* pass_list, } SolidColorDrawQuad* AddQuad(TestRenderPass* pass, - gfx::Rect rect, + const gfx::Rect& rect, SkColor color) { MockQuadCuller quad_sink(&pass->quad_list, &pass->shared_quad_state_list); AppendQuadsData data(pass->id); @@ -51,7 +51,7 @@ SolidColorDrawQuad* AddQuad(TestRenderPass* pass, } SolidColorDrawQuad* AddClippedQuad(TestRenderPass* pass, - gfx::Rect rect, + const gfx::Rect& rect, SkColor color) { MockQuadCuller quad_sink(&pass->quad_list, &pass->shared_quad_state_list); AppendQuadsData data(pass->id); @@ -72,7 +72,7 @@ SolidColorDrawQuad* AddClippedQuad(TestRenderPass* pass, } SolidColorDrawQuad* AddTransformedQuad(TestRenderPass* pass, - gfx::Rect rect, + const gfx::Rect& rect, SkColor color, const gfx::Transform& transform) { MockQuadCuller quad_sink(&pass->quad_list, &pass->shared_quad_state_list); |