Re-enable main-frame and subframe CSSOM smooth scroll layout tests

Re-enable main-frame and subframe CSSOM smooth scroll layout tests

This re-enables tests that were disabled when the ScrollTopLeftInterop
runtime-enabled feature was disabled for tests in http://crrev.com/891523002.

Tests that previously scrolled the document element now scroll the body
element, and FrameView::setScrollPosition now checks the scroll-behavior
property on the body element rather than on the document element.

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

[ajuma, 2015-02-03 19:12:57]

ajuma@chromium.org changed reviewers:
+ jochen@chromium.org, skobes@chromium.org

[skobes, 2015-02-03 19:32:58]

lgtm

[ajuma, 2015-02-03 19:34:00]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2015-02-03 19:34:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/882153004/1

[commit-bot: I haz the power, 2015-02-03 19:41:28]

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

[commit-bot: I haz the power, 2015-02-03 19:41:28]

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

[ajuma, 2015-02-03 19:42:22]

jochen, PTAL for core/ OWNERS.

[jochen (gone - plz use gerrit), 2015-02-04 12:48:50]

lgtm

[jochen (gone - plz use gerrit), 2015-02-04 12:49:17]

https://codereview.chromium.org/882153004/diff/1/Source/core/frame/FrameView.cpp
File Source/core/frame/FrameView.cpp (right):

https://codereview.chromium.org/882153004/diff/1/Source/core/frame/FrameView....
Source/core/frame/FrameView.cpp:1556: RenderObject* renderer =
m_frame->document()->body() ? m_frame->document()->body()->renderer() : 0;
should this maybe depend on the ScrollTopLeftInteropt runtime feature?

[ajuma, 2015-02-04 14:40:24]

New patchsets have been uploaded after l-g-t-m from
skobes@chromium.org,jochen@chromium.org

[ajuma, 2015-02-04 15:08:11]

https://codereview.chromium.org/882153004/diff/1/Source/core/frame/FrameView.cpp
File Source/core/frame/FrameView.cpp (right):

https://codereview.chromium.org/882153004/diff/1/Source/core/frame/FrameView....
Source/core/frame/FrameView.cpp:1556: RenderObject* renderer =
m_frame->document()->body() ? m_frame->document()->body()->renderer() : 0;
On 2015/02/04 12:49:17, jochen (slow) wrote:
> should this maybe depend on the ScrollTopLeftInteropt runtime feature?

Good point, done.

[ajuma, 2015-02-04 15:16:12]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2015-02-04 15:16:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/882153004/60001

[commit-bot: I haz the power, 2015-02-04 17:03:17]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=189512