Remove RenderLayer::collectFragments.

Remove RenderLayer::collectFragments.

Sky always has exactly one per RenderLayer. This patch
gets rid of the hitTestLayer use of LayerFragment
and gets paintLayerContents to create the list of
fragments itself since it's the only caller.

Also, delete dead code from LayerFragment.h.

R=abarth@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/4a3b676dc3387b64575d4a407c90fa8a6f466f4c

[ojan, 2014-12-08 21:20:54]

ojan@chromium.org changed reviewers:
+ eseidel@chromium.org, esprehn@chromium.org, rafaelw@chromium.org

[abarth-chromium, 2014-12-08 23:34:00]

abarth@chromium.org changed reviewers:
+ abarth@chromium.org

[abarth-chromium, 2014-12-08 23:34:00]

lgtm

[ojan, 2014-12-09 00:37:58]

Committed patchset #1 (id:1) manually as
4a3b676dc3387b64575d4a407c90fa8a6f466f4c.

[eseidel, 2014-12-10 21:43:50]

https://codereview.chromium.org/778043005/diff/1/sky/engine/core/rendering/Re...
File sky/engine/core/rendering/RenderLayer.cpp (right):

https://codereview.chromium.org/778043005/diff/1/sky/engine/core/rendering/Re...
sky/engine/core/rendering/RenderLayer.cpp:1572: if (isSelfPaintingLayer() &&
!foregroundRect.intersects(hitTestLocation)) {
This ! looks wrong.

[eseidel, 2014-12-10 22:42:25]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/788383002/ by eseidel@chromium.org.

The reason for reverting is: This broke hit testing.  Hit testing always returns
the root node now. :).