It's better to open the test page on low dpi devices because there is a bug on high dpi devices ( issue 840478 ) right now which affects the result.

 Issue 841558  has been merged into this issue.

 Issue 840478  has been merged into this issue.

See crrev.com/c/1054488 for proposed fix.

This issue is marked as a release blocker with no OS labels associated. Please add an appropriate OS label.

All release blocking issues should have OS labels associated to it, so that the issue can tracked and promptly verified, once it gets fixed.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d5270f6e22964848928f2f6a6d2263d012186e24

commit d5270f6e22964848928f2f6a6d2263d012186e24
Author: Yi Gu <yigu@chromium.org>
Date: Mon Jun 04 22:40:02 2018

Compute correct sticky box constraints within scrollable containers

Currently calculating the skipped offset between sticky and its
containing block assumes that the latter is not scrollable. However,
when it is we should ignore the scroll offset from the container
otherwise scrolling the container would cause the sticky offset
incorrectly double accumulated.

Bug:  841551 
Change-Id: Ib49aaa74808847dcf6fce7dc0db9ac4d2de06c64
Reviewed-on: https://chromium-review.googlesource.com/1054488
Commit-Queue: Yi Gu <yigu@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564276}
[add] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/WebKit/LayoutTests/external/wpt/css/css-position/position-sticky-hyperlink-ref.html
[add] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/WebKit/LayoutTests/external/wpt/css/css-position/position-sticky-hyperlink.html
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_box.cc
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_box.h
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_box_model_object.cc
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_box_model_object_test.cc
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_inline.cc
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_inline.h
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_object.cc
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_object.h
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_table_cell.cc
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/layout_table_cell.h
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/map_coordinates_flags.h
[modify] https://crrev.com/d5270f6e22964848928f2f6a6d2263d012186e24/third_party/blink/renderer/core/layout/map_coordinates_test.cc

position : sticky has stopped working since https://chromium.googlesource.com/chromium/src/+/b087bb812734db76b31de8c6e2efd62d1b0a15a7. That patch exposed an existing bug due to incorrect computation of sticky box constraints within scrollable containers. The fix has been landed via #8.

Another test case just FYI: https://developer.mozilla.org/en-US/docs/Web/CSS/position. Click "position: sticky" and scroll the panel on the right.

Your change meets the bar and is auto-approved for M68. Please go ahead and merge the CL to branch 3440 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), kariahda@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/46de48d287c6eaf2f811b47312e3f2f0d9532e01

commit 46de48d287c6eaf2f811b47312e3f2f0d9532e01
Author: Yi Gu <yigu@chromium.org>
Date: Wed Jun 06 13:29:23 2018

Compute correct sticky box constraints within scrollable containers

Currently calculating the skipped offset between sticky and its
containing block assumes that the latter is not scrollable. However,
when it is we should ignore the scroll offset from the container
otherwise scrolling the container would cause the sticky offset
incorrectly double accumulated.

Bug:  841551 
Change-Id: Ib49aaa74808847dcf6fce7dc0db9ac4d2de06c64
Reviewed-on: https://chromium-review.googlesource.com/1054488
Commit-Queue: Yi Gu <yigu@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#564276}(cherry picked from commit d5270f6e22964848928f2f6a6d2263d012186e24)
Reviewed-on: https://chromium-review.googlesource.com/1087333
Reviewed-by: Yi Gu <yigu@chromium.org>
Cr-Commit-Position: refs/branch-heads/3440@{#209}
Cr-Branched-From: 010ddcfda246975d194964ccf20038ebbdec6084-refs/heads/master@{#561733}
[add] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/WebKit/LayoutTests/external/wpt/css/css-position/position-sticky-hyperlink-ref.html
[add] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/WebKit/LayoutTests/external/wpt/css/css-position/position-sticky-hyperlink.html
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_box.cc
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_box.h
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_box_model_object.cc
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_box_model_object_test.cc
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_inline.cc
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_inline.h
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_object.cc
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_object.h
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_table_cell.cc
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/layout_table_cell.h
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/map_coordinates_flags.h
[modify] https://crrev.com/46de48d287c6eaf2f811b47312e3f2f0d9532e01/third_party/blink/renderer/core/layout/map_coordinates_test.cc

Able to reproduce the issue on Mac 10.13.3 using chrome build without fix.

Verified the fix on Mac 10.13.3, Win-10 and Ubuntu 17.10 using Chrome beta version #68.0.3440.17 as per the merged issue id: 841558 in comment #2.
Attaching screen cast for reference.
Observed that clicking a sticky hyperlink after scroll navigated to the new page.
Hence, the fix is working as expected. 
Adding the verified labels.

Thanks...!!

Deleted: 841551.mp4 166 KB

	841551.mp4 166 KB View Download

 Issue 851832  has been merged into this issue.

 Issue 856448  has been merged into this issue.