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

Unified Diff: third_party/WebKit/LayoutTests/fast/overflow/overflow-y-scroll.html

Issue 1475863005: [Async][WIP] Call FrameLoader::checkCompleted() asynchronously to avoid sync body.onload() Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. 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/fast/overflow/overflow-y-scroll.html
diff --git a/third_party/WebKit/LayoutTests/fast/overflow/overflow-y-scroll.html b/third_party/WebKit/LayoutTests/fast/overflow/overflow-y-scroll.html
index 73e66538e6b5a4a343a772826df3145eba102f00..e0501665a0f481a2da8420068217aa50b041feb1 100644
--- a/third_party/WebKit/LayoutTests/fast/overflow/overflow-y-scroll.html
+++ b/third_party/WebKit/LayoutTests/fast/overflow/overflow-y-scroll.html
@@ -2,8 +2,10 @@
<head>
<title>Bug 25313 : Missing scrollbars in GMail</title>
<script>
- if (window.testRunner)
+ if (window.testRunner) {
window.testRunner.dumpAsText();
+ window.testRunner.waitUntilDone();
+ }
function inject()
{
@@ -24,6 +26,9 @@
injectee.style.width = "400px";
// measure to see if there is a scrollbar. Pass if there is.
document.getElementById("test").innerHTML = doc.body.offsetWidth < 400 ? "TEST PASSED" : "TEST FAILED";
+ if (window.testRunner) {
+ window.testRunner.notifyDone();
+ }
}
</script>
</head>

Powered by Google App Engine
This is Rietveld 408576698