ScrollTopLeftInterop should not be marked experimental

ScrollTopLeftInterop should not be marked experimental

The feature breaks a lot of sites (e.g. those using closure) and it's
not clear what the path forward for it is. Until that is clear, we
should not mark it as experimental.

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

[jochen (gone - plz use gerrit), 2015-02-02 13:59:09]

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

[jochen (gone - plz use gerrit), 2015-02-02 13:59:10]

please review.

The motivation for this change is that it makes running chrome with experimental
features enabled almost impossible because UI elements are jumping around
randomly.

I don't really see how we can turn on this feature given that everybody works
around chrome's behavior, so we shouldn't use that for testing either.

Note that I had to disable a bunch of scroll-behavior/ tests as the either time
out (when scrolling documentElement) or hit supposedly unreachable code (when
scrolling body) here
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

[ajuma, 2015-02-02 14:29:38]

On 2015/02/02 13:59:10, jochen (slow) wrote:
> please review.
> 
> The motivation for this change is that it makes running chrome with
experimental
> features enabled almost impossible because UI elements are jumping around
> randomly.
> 
> I don't really see how we can turn on this feature given that everybody works
> around chrome's behavior, so we shouldn't use that for testing either.
> 
> Note that I had to disable a bunch of scroll-behavior/ tests as the either
time
> out (when scrolling documentElement) or hit supposedly unreachable code (when
> scrolling body) here
>
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

lgtm. It's not immediately obvious to me why we're hitting that code while
scrolling body, but I'll investigate further after this lands.

[ajuma, 2015-02-02 14:29:53]

One question:

https://codereview.chromium.org/891523002/diff/100001/LayoutTests/fast/scroll...
File LayoutTests/fast/scrolling/hover-during-scroll.html (right):

https://codereview.chromium.org/891523002/diff/100001/LayoutTests/fast/scroll...
LayoutTests/fast/scrolling/hover-during-scroll.html:41:
shouldBe("document.documentElement.scrollTop", "0");
Does using the body element here work? (Also below.)

[jochen (gone - plz use gerrit), 2015-02-02 19:45:23]

On 2015/02/02 at 14:29:53, ajuma wrote:
> One question:
> 
>
https://codereview.chromium.org/891523002/diff/100001/LayoutTests/fast/scroll...
> File LayoutTests/fast/scrolling/hover-during-scroll.html (right):
> 
>
https://codereview.chromium.org/891523002/diff/100001/LayoutTests/fast/scroll...
> LayoutTests/fast/scrolling/hover-during-scroll.html:41:
shouldBe("document.documentElement.scrollTop", "0");
> Does using the body element here work? (Also below.)

appears to work, updated

[jochen (gone - plz use gerrit), 2015-02-02 20:30:48]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2015-02-02 20:31:58]

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

[commit-bot: I haz the power, 2015-02-02 21:11:14]

Committed patchset #7 (id:120001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=189364