Get rid of ScrollView.

Get rid of ScrollView.

We only allow overflow scrolling. The frame isn't special.
This is a first step in making that happen. There's a lot of
code to remove after this patch, but this gets rid of
ScrollView and a bunch of frame-level scrolling code.

Had to add in a FrameWidget class so that Scrollbar.cpp had
a way of getting to FrameView::removeChild without pulling
a core class into platform. This might go away when we rip
out the Widget tree if we made it so that FrameView didn't
keep a list of Scrollbar instances.

Modified scrollbar.html to use overflow scrolling instead of
frame level scrolling. Once we get rid of the split between
Document and documentElement, we'll be able to make the root
element in the page scrollable as well (i.e. any child of the
Document).

R=abarth@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/352610817aa32b4ac7d2237f8b4cfa8e20db64ea

[ojan, 2014-10-24 02:43:19]

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

[ojan, 2014-10-24 02:43:20]

Sorry this is so enormous.

[abarth-chromium, 2014-10-24 03:16:06]

LGTM

This means we can remove fixed position too, right?

https://codereview.chromium.org/646273006/diff/20001/sky/engine/core/frame/Fr...
File sky/engine/core/frame/FrameView.cpp (right):

https://codereview.chromium.org/646273006/diff/20001/sky/engine/core/frame/Fr...
sky/engine/core/frame/FrameView.cpp:1865: current != end; ++current) {
for (const auto& widget : m_children) {
  ...
}

For realz

[ojan, 2014-10-24 03:20:30]

Committed patchset #2 (id:20001) manually as
352610817aa32b4ac7d2237f8b4cfa8e20db64ea (presubmit successful).

[ojan, 2014-10-24 03:21:01]

https://codereview.chromium.org/646273006/diff/20001/sky/engine/core/frame/Fr...
File sky/engine/core/frame/FrameView.cpp (right):

https://codereview.chromium.org/646273006/diff/20001/sky/engine/core/frame/Fr...
sky/engine/core/frame/FrameView.cpp:1865: current != end; ++current) {
On 2014/10/24 03:16:06, abarth wrote:
> for (const auto& widget : m_children) {
>   ...
> }
> 
> For realz

Amazing. Done.

I guess it's time to actually learn C++11 syntax!