Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(109)

Unified Diff: third_party/WebKit/LayoutTests/fast/css/sticky/sticky-container-moved.html

Issue 2165763002: Fix layout tests that would fail if ScrollAnchoring was promoted to experimental (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Undo promoting ScrollAnchoring to test Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/css/sticky/sticky-container-moved.html
diff --git a/third_party/WebKit/LayoutTests/fast/css/sticky/sticky-container-moved.html b/third_party/WebKit/LayoutTests/fast/css/sticky/sticky-container-moved.html
index b2bfc9623ad3233d4d2a49fae5d5391dce6a2c3f..9f3960f97a69b26dfafbc0ec5cf77cb8e9b5488d 100644
--- a/third_party/WebKit/LayoutTests/fast/css/sticky/sticky-container-moved.html
+++ b/third_party/WebKit/LayoutTests/fast/css/sticky/sticky-container-moved.html
@@ -2,7 +2,7 @@
<script>
if (window.internals) {
internals.settings.setCSSStickyPositionEnabled(true);
-}
+}
</script>
<style>
@@ -10,6 +10,7 @@ body {
margin: 0;
height: 2000px;
overflow: hidden; /* hide scrollbars */
+ overflow-anchor: none;
ymalik 2016/07/22 16:53:24 @skobes, I couldn't come up with a way to modify t
skobes 2016/07/25 20:46:38 How about calling scrollTo after changing height,
ymalik 2016/07/26 21:03:50 I think the purpose of the test is to verify that
skobes 2016/07/26 21:19:14 Good point. Using the opt out here is fine.
}
#spacer {

Powered by Google App Engine
This is Rietveld 408576698