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

Unified Diff: third_party/WebKit/LayoutTests/css3/flexbox/overflow-auto-resizes-correctly.html

Issue 1734203002: Fully (?) fix overflow: auto with delayed scroll updates (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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/css3/flexbox/overflow-auto-resizes-correctly.html
diff --git a/third_party/WebKit/LayoutTests/css3/flexbox/overflow-auto-resizes-correctly.html b/third_party/WebKit/LayoutTests/css3/flexbox/overflow-auto-resizes-correctly.html
index b031fb30ae81b686f6d9e13ce188444d1e471386..436b2d13b5afdc08c9175da6151030ae562f2618 100644
--- a/third_party/WebKit/LayoutTests/css3/flexbox/overflow-auto-resizes-correctly.html
+++ b/third_party/WebKit/LayoutTests/css3/flexbox/overflow-auto-resizes-correctly.html
@@ -19,6 +19,7 @@
}
.vbox {
overflow-y: auto;
+ max-height: 200px;
cbiesinger 2016/02/25 20:46:26 I made this change because I am no longer sure tha
}
.intrinsic-height-box {
min-height: -webkit-min-content;
@@ -76,11 +77,9 @@
var scrollbarSize = measure.offsetWidth - measure.clientWidth;
shouldNotBe("scrollbarSize", "0");
- // TODO(cbiesinger): These should really be 100 and 100 + scrollbarSize, but
- // we currently have a bug that makes it not so.
var inner = document.getElementById('inner');
- shouldBe("inner.clientWidth", "100 - scrollbarSize");
- shouldBe("inner.offsetWidth", "100");
+ shouldBe("inner.clientWidth", "100");
+ shouldBe("inner.offsetWidth", "100 + scrollbarSize");
var vbox = document.querySelector('.vbox');
shouldBe("vbox.clientWidth", "100");

Powered by Google App Engine
This is Rietveld 408576698