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

Unified Diff: third_party/WebKit/LayoutTests/dialog/scrollable-after-close.html

Issue 2437303002: Fix unscrollable scrollers after closing a <dialog>. (Closed)
Patch Set: address review comments Created 4 years, 2 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/dialog/scrollable-after-close-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/dialog/scrollable-after-close.html
diff --git a/third_party/WebKit/LayoutTests/dialog/scrollable-after-close.html b/third_party/WebKit/LayoutTests/dialog/scrollable-after-close.html
new file mode 100644
index 0000000000000000000000000000000000000000..567fba734960fbdd2527ed665e6b228441f7af0a
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/dialog/scrollable-after-close.html
@@ -0,0 +1,35 @@
+<!DOCTYPE html>
+<script src="../resources/js-test.js"></script>
+<style>
+html, body { height: 100%; margin: 0; overflow: hidden; }
+#scroller {
+ overflow: scroll;
+ position: relative;
+ z-index: 0;
+ width: 300px;
+ height: 50%;
+ left: 20px;
+ top: 20px;
+ border: 1px solid black;
+}
+</style>
+<dialog></dialog>
+<div id="scroller">
+ <div style="height: 700px; width: 700px"></div>
+</div>
+<script>
+
+description("Tests that FrameView::m_scrollableAreas is updated after " +
+ "closing a modal dialog when the scroller and its ancestors " +
+ "have percentage heights. (see crbug.com/633520)");
+
+var dialog = document.querySelector("dialog");
+dialog.showModal();
+dialog.close();
+
+// Force layout.
+document.body.offsetWidth;
+
+shouldBe("internals.numberOfScrollableAreas(document)", "1");
+
+</script>
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/dialog/scrollable-after-close-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698