Scroll sticky elements into view.

Scroll sticky elements into view.

We specifically ignore elements that scroll with the viewport, but sticky
elements should be scrolled into view.

BUG=662723
TEST=fast/scroll-behavior/scroll-into-view-sticky.html

Committed: https://crrev.com/9e23825d88c48013fb892c755106d21c9aebeeb2
Cr-Commit-Position: refs/heads/master@{#430969}

[flackr, 2016-11-09 13:00:44]

flackr@chromium.org changed reviewers:
+ bokan@chromium.org

[flackr, 2016-11-09 13:00:45]

Hey,

It looks like we avoided trying to scroll to position fixed because of course it
would stay where it was in the viewport, but we should not do the same for
position sticky because it is constrained. There is another issue here, which is
that we should probably scroll to the "unstuck" position which would avoid
scrolling to a different position if you tried to scroll to it again, but that
is not consistent with what Firefox and Safari do so for now I'd like to just
scroll to its current position and push on that one in other browsers as well.

PTAL, thanks.

[bokan, 2016-11-09 14:26:09]

Agree matching other browsers for now is best. lgtm.

[flackr, 2016-11-09 14:31:05]

The CQ bit was checked by flackr@chromium.org

[commit-bot: I haz the power, 2016-11-09 14:31:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-09 17:54:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-09 18:02:57]

Description was changed from

==========
Scroll sticky elements into view.

We specifically ignore elements that scroll with the viewport, but sticky
elements should be scrolled into view.

BUG=662723
TEST=fast/scroll-behavior/scroll-into-view-sticky.html
==========

to

==========
Scroll sticky elements into view.

We specifically ignore elements that scroll with the viewport, but sticky
elements should be scrolled into view.

BUG=662723
TEST=fast/scroll-behavior/scroll-into-view-sticky.html

Committed: https://crrev.com/9e23825d88c48013fb892c755106d21c9aebeeb2
Cr-Commit-Position: refs/heads/master@{#430969}
==========

[commit-bot: I haz the power, 2016-11-09 18:02:57]

Patchset 1 (id:??) landed as
https://crrev.com/9e23825d88c48013fb892c755106d21c9aebeeb2
Cr-Commit-Position: refs/heads/master@{#430969}