cc: Chromium-stylify the DelegatedRendererLayer classes and tests.

cc: Chromium-stylify the DelegatedRendererLayer classes and tests.

No change in behaviour, style change only.

Depends on: https://codereview.chromium.org/12040011

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=178226

[piman, 2013-01-22 18:52:23]

LGTM+nit

https://codereview.chromium.org/12041012/diff/3008/cc/delegated_renderer_laye...
File cc/delegated_renderer_layer_impl.cc (right):

https://codereview.chromium.org/12041012/diff/3008/cc/delegated_renderer_laye...
cc/delegated_renderer_layer_impl.cc:204: else {
nit: need brackets around first clause if second one does
(http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Condit...
)

[danakj, 2013-01-22 19:00:09]

https://codereview.chromium.org/12041012/diff/3008/cc/delegated_renderer_laye...
File cc/delegated_renderer_layer_impl.cc (right):

https://codereview.chromium.org/12041012/diff/3008/cc/delegated_renderer_laye...
cc/delegated_renderer_layer_impl.cc:204: else {
On 2013/01/22 18:52:23, piman wrote:
> nit: need brackets around first clause if second one does
>
(http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Condit...
> )

Ah, thanks. Done.

[commit-bot: I haz the power, 2013-01-22 23:06:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/danakj@chromium.org/12041012/6001

[commit-bot: I haz the power, 2013-01-23 01:30:05]

Retried try job too often on win_rel for step(s) content_browsertests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-01-23 02:08:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/danakj@chromium.org/12041012/6001

[commit-bot: I haz the power, 2013-01-23 03:22:29]

Change committed as 178226