Hide RenderLayer z-order and normal flow lists behind iterators

[Simon Fraser (smfr)]

Hide RenderLayer z-order and normal flow lists behind iterators

[Simon Fraser (smfr)]

Created attachment 352011 [details]
Patch

[zalan]

Comment on attachment 352011 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=352011&action=review

> Source/WebCore/rendering/RenderLayer.h:373
> +        iterator begin() { return m_layerList ? m_layerList->begin() : nullptr; }
> +        iterator end() { return m_layerList ? m_layerList->end() : nullptr; }
> +
> +        reverse_iterator rbegin() { return reverse_iterator(end()); }
> +        reverse_iterator rend() { return reverse_iterator(begin()); }

It's a bit odd that a class named Iterator has functions like these.

[Simon Fraser (smfr)]

LayerIterable? LayerList?

[zalan]

(In reply to Simon Fraser (smfr) from comment #3)
> LayerIterable? LayerList?

LayerList is fine.

[Simon Fraser (smfr)]

https://trac.webkit.org/changeset/237058/webkit

[Radar WebKit Bug Importer]

<rdar://problem/45874621>