Remove unneeded break in CompositedLayerMapping::containingSquashedLayer

Remove unneeded break in CompositedLayerMapping::containingSquashedLayer

Remove unneeded break because the return makes it unreachable.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178357

[rwlbuis, 2014-07-16 19:42:42]

PTAL

[Ian Vollick, 2014-07-16 19:45:45]

On 2014/07/16 19:42:42, rwlbuis wrote:
> PTAL

lgtm.

[Ian Vollick, 2014-07-16 19:47:45]

On 2014/07/16 19:45:45, Ian Vollick wrote:
> On 2014/07/16 19:42:42, rwlbuis wrote:
> > PTAL
> 
> lgtm.

Actually, two nits :)

Can you put curlies around the for (since it's got more than 1 line in it) and
remove the redundant stuff in the description?

[rwlbuis, 2014-07-16 19:54:07]

On 2014/07/16 19:47:45, Ian Vollick wrote:
> On 2014/07/16 19:45:45, Ian Vollick wrote:
> > On 2014/07/16 19:42:42, rwlbuis wrote:
> > > PTAL
> > 
> > lgtm.
> 
> Actually, two nits :)
> 
> Can you put curlies around the for (since it's got more than 1 line in it) and
> remove the redundant stuff in the description?

Yeah I was a bit too hasty with my patch :) I hope I fixed it now.

[Ian Vollick, 2014-07-16 19:56:03]

On 2014/07/16 19:54:07, rwlbuis wrote:
> On 2014/07/16 19:47:45, Ian Vollick wrote:
> > On 2014/07/16 19:45:45, Ian Vollick wrote:
> > > On 2014/07/16 19:42:42, rwlbuis wrote:
> > > > PTAL
> > > 
> > > lgtm.
> > 
> > Actually, two nits :)
> > 
> > Can you put curlies around the for (since it's got more than 1 line in it)
and
> > remove the redundant stuff in the description?
> 
> Yeah I was a bit too hasty with my patch :) I hope I fixed it now.

Thanks! slgtm.

[rwlbuis, 2014-07-16 20:01:13]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-07-16 20:02:04]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rob.buis@samsung.com/392403003/20001

[commit-bot: I haz the power, 2014-07-16 21:07:43]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/17339)

[commit-bot: I haz the power, 2014-07-16 22:03:10]

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

[commit-bot: I haz the power, 2014-07-16 22:03:11]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/17348)

[rwlbuis, 2014-07-17 09:38:09]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-07-17 09:38:22]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rob.buis@samsung.com/392403003/20001

[commit-bot: I haz the power, 2014-07-17 11:10:42]

Change committed as 178357