diff options
| author | prashant.n@samsung.com <prashant.n@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-28 06:37:17 +0000 |
|---|---|---|
| committer | prashant.n@samsung.com <prashant.n@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-28 06:37:17 +0000 |
| commit | c93cbb62f684183e0e5689a9fa35619daac10f7b (patch) | |
| tree | 8002e049a31ad5cad79f63d19fe1f79b68b8ae2f /cc/test/fake_picture_layer_tiling_client.h | |
| parent | 22a9610ffaed4d7690463d2f6f710d46c206c70f (diff) | |
| download | chromium_src-c93cbb62f684183e0e5689a9fa35619daac10f7b.zip chromium_src-c93cbb62f684183e0e5689a9fa35619daac10f7b.tar.gz chromium_src-c93cbb62f684183e0e5689a9fa35619daac10f7b.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
Review URL: https://codereview.chromium.org/145313006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247426 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/test/fake_picture_layer_tiling_client.h')
| -rw-r--r-- | cc/test/fake_picture_layer_tiling_client.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cc/test/fake_picture_layer_tiling_client.h b/cc/test/fake_picture_layer_tiling_client.h index 2e4e9d6..6383a22 100644 --- a/cc/test/fake_picture_layer_tiling_client.h +++ b/cc/test/fake_picture_layer_tiling_client.h @@ -25,9 +25,9 @@ class FakePictureLayerTilingClient : public PictureLayerTilingClient { PictureLayerTiling* tiling, const gfx::Rect& rect) OVERRIDE; virtual void UpdatePile(Tile* tile) OVERRIDE {} virtual gfx::Size CalculateTileSize( - gfx::Size content_bounds) const OVERRIDE; + const gfx::Size& content_bounds) const OVERRIDE; - void SetTileSize(gfx::Size tile_size); + void SetTileSize(const gfx::Size& tile_size); gfx::Size TileSize() const { return tile_size_; } scoped_refptr<PicturePileImpl> pile() { return pile_; } const PicturePileImpl* pile() const { return pile_.get(); } |