cc: Clean up iterator template to only take 1 parameter.

cc: Clean up iterator template to only take 1 parameter.

LayerIterator takes 4 parameters. One is the FrontToBack ordering
but that is now the only option. The other 3 are for the type of layers,
render surfaces, and the render surface layer list. The latter two are
now already available as typedefs off of Layer and LayerImpl, so we
only need Layer/LayerImpl in the template arguments.

R=enne@chromium.org, enne
BUG=YAK

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

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

[enne (OOO), 2014-02-11 23:25:22]

lgtm

https://codereview.chromium.org/156603005/diff/30001/cc/debug/debug_rect_hist...
File cc/debug/debug_rect_history.cc (right):

https://codereview.chromium.org/156603005/diff/30001/cc/debug/debug_rect_hist...
cc/debug/debug_rect_history.cc:115: MathUtil::MapClippedRect(
This is just a drive-by? Maybe stick it in a different patch?

[danakj, 2014-02-11 23:31:00]

https://codereview.chromium.org/156603005/diff/30001/cc/debug/debug_rect_hist...
File cc/debug/debug_rect_history.cc (right):

https://codereview.chromium.org/156603005/diff/30001/cc/debug/debug_rect_hist...
cc/debug/debug_rect_history.cc:115: MathUtil::MapClippedRect(
On 2014/02/11 23:25:22, enne wrote:
> This is just a drive-by? Maybe stick it in a different patch?

This was not meant to be here.. it was a mistake in my other CL that I fixed.
Thanks for spotting it.

[danakj, 2014-02-14 00:07:44]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2014-02-14 00:08:16]

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

[commit-bot: I haz the power, 2014-02-14 02:54:15]

Change committed as 251238

[danakj, 2014-02-27 23:13:37]

Uploaded a new CL here with an empty struct member added to the LayerIterator
class to give the class a size of 24bytes as it had before this CL happened,
instead of 16 bytes (on 64bit linux).

[ajwong, 2014-02-28 02:39:25]

Hey Dana,

Can you land this? We'd like to test things out.

-Albert


On Thu, Feb 27, 2014 at 3:13 PM, <danakj@chromium.org> wrote:

> Uploaded a new CL here with an empty struct member added to the
> LayerIterator
> class to give the class a size of 24bytes as it had before this CL
> happened,
> instead of 16 bytes (on 64bit linux).
>
> https://codereview.chromium.org/156603005/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[danakj, 2014-02-28 16:44:55]

Committed patchset #4 manually as r254156 (presubmit successful).