diff options
| author | prashant.n@samsung.com <prashant.n@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-29 22:58:26 +0000 |
|---|---|---|
| committer | prashant.n@samsung.com <prashant.n@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-29 22:58:26 +0000 |
| commit | 64348eae94a2d07e923aaa0262b6c47947ffa3b3 (patch) | |
| tree | c75d0c39b289b4a2543426bbe4461cc3e7117c46 /cc/test/fake_picture_layer_impl.h | |
| parent | 44ea5317cf5920949c483be09f2931ae20cf23d1 (diff) | |
| download | chromium_src-64348eae94a2d07e923aaa0262b6c47947ffa3b3.zip chromium_src-64348eae94a2d07e923aaa0262b6c47947ffa3b3.tar.gz chromium_src-64348eae94a2d07e923aaa0262b6c47947ffa3b3.tar.bz2 | |
[#7] Pass gfx::Size by const ref.
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 and some
other scenarios mentioned in the bug.
BUG=159273
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247426
Review URL: https://codereview.chromium.org/145313006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247769 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/test/fake_picture_layer_impl.h')
| -rw-r--r-- | cc/test/fake_picture_layer_impl.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/cc/test/fake_picture_layer_impl.h b/cc/test/fake_picture_layer_impl.h index a289934..477b6b9 100644 --- a/cc/test/fake_picture_layer_impl.h +++ b/cc/test/fake_picture_layer_impl.h @@ -26,7 +26,8 @@ class FakePictureLayerImpl : public PictureLayerImpl { OVERRIDE; virtual void AppendQuads(QuadSink* quad_sink, AppendQuadsData* append_quads_data) OVERRIDE; - virtual gfx::Size CalculateTileSize(gfx::Size content_bounds) const OVERRIDE; + virtual gfx::Size CalculateTileSize( + const gfx::Size& content_bounds) const OVERRIDE; using PictureLayerImpl::AddTiling; using PictureLayerImpl::CleanUpTilingsOnActiveLayer; @@ -54,7 +55,7 @@ class FakePictureLayerImpl : public PictureLayerImpl { const Region& invalidation() const { return invalidation_; } void set_invalidation(const Region& region) { invalidation_ = region; } - void set_fixed_tile_size(gfx::Size size) { fixed_tile_size_ = size; } + void set_fixed_tile_size(const gfx::Size& size) { fixed_tile_size_ = size; } void CreateDefaultTilingsAndTiles(); void SetAllTilesVisible(); |