cc: Pass SurfaceId by const ref in more places

cc: Pass SurfaceId by const ref in more places

SurfaceId is 128-bits and so it cannot fit in a single register.
Thus passing it around by const reference is potentially more
efficient, and more consistent with how we pass around other non-trivial
structs such as gfx::Rect.

BUG=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/2c5cb07a5bb810d82e021a26d8cba2bb5664420e
Cr-Commit-Position: refs/heads/master@{#405267}

[Fady Samuel, 2016-07-13 17:15:13]

Description was changed from

==========
cc: Pass SurfaceId by const ref in more places

SurfaceId is 128-bits and so it cannot fit in a single register.
Thus passing it around by const reference is potentially more
efficient, and more consistent with how we pass around other non-trivial
structs such as gfx::Rect.

BUG=none
==========

to

==========
cc: Pass SurfaceId by const ref in more places

SurfaceId is 128-bits and so it cannot fit in a single register.
Thus passing it around by const reference is potentially more
efficient, and more consistent with how we pass around other non-trivial
structs such as gfx::Rect.

BUG=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Fady Samuel, 2016-07-13 17:18:40]

fsamuel@chromium.org changed reviewers:
+ jam@chromium.org, weiliangc@chromium.org

[Fady Samuel, 2016-07-13 17:18:41]

+weilangc@ for cc
+jam@ for content.

[Fady Samuel, 2016-07-13 18:18:46]

The CQ bit was checked by fsamuel@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-13 18:19:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[weiliangc, 2016-07-13 18:30:53]

LGTM

[commit-bot: I haz the power, 2016-07-13 19:15:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-13 19:15:28]

Dry run: This issue passed the CQ dry run.

[jam, 2016-07-13 19:20:12]

lgtm

[Fady Samuel, 2016-07-13 19:20:45]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2016-07-13 19:23:12]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-13 19:34:13]

Your CL relies on deprecated CQ feature(s):
 * Specifying master names without "master." prefix is deprecated:
tryserver.blink. For more details, see http://crbug.com/617627.

[commit-bot: I haz the power, 2016-07-13 20:19:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-13 20:22:11]

Description was changed from

==========
cc: Pass SurfaceId by const ref in more places

SurfaceId is 128-bits and so it cannot fit in a single register.
Thus passing it around by const reference is potentially more
efficient, and more consistent with how we pass around other non-trivial
structs such as gfx::Rect.

BUG=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Pass SurfaceId by const ref in more places

SurfaceId is 128-bits and so it cannot fit in a single register.
Thus passing it around by const reference is potentially more
efficient, and more consistent with how we pass around other non-trivial
structs such as gfx::Rect.

BUG=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/2c5cb07a5bb810d82e021a26d8cba2bb5664420e
Cr-Commit-Position: refs/heads/master@{#405267}
==========

[commit-bot: I haz the power, 2016-07-13 20:22:12]

Patchset 1 (id:??) landed as
https://crrev.com/2c5cb07a5bb810d82e021a26d8cba2bb5664420e
Cr-Commit-Position: refs/heads/master@{#405267}