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

Unified Diff: LayoutTests/fast/media/media-query-list-listener-ordering.html

Issue 337883003: Call media query change listeners asynchronously. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: fix typo Created 6 years, 6 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: LayoutTests/fast/media/media-query-list-listener-ordering.html
diff --git a/LayoutTests/fast/media/media-query-list-listener-ordering.html b/LayoutTests/fast/media/media-query-list-listener-ordering.html
index 0394116168f0fc33c51baa69ba019f221ef2f1f7..2cc95a7d12d70e89ac63f1ce0141a8da0b36decb 100644
--- a/LayoutTests/fast/media/media-query-list-listener-ordering.html
+++ b/LayoutTests/fast/media/media-query-list-listener-ordering.html
@@ -5,6 +5,7 @@
<script>
description("Media query listeners should fire grouped by the MediaQueryList.");
+ var jsTestIsAsync = true;
var sandbox = document.getElementById("sandbox");
var iframe = document.createElement("iframe");
@@ -25,12 +26,13 @@
mediaList2.addListener(makeListener("mediaList2_1"));
mediaList1.addListener(makeListener("mediaList1_1"));
mediaList2.addListener(makeListener("mediaList2_2"));
-
- // FIXME: You shouldn't need to touch offsetTop to force these updates.
+ mediaList3.addListener(verifyResult);
iframe.style.width = "200px";
- iframe.offsetTop;
var expectedResult = ["mediaList1_1", "mediaList2_1", "mediaList2_2", "mediaList3_1"];
- shouldBe("listenersCalled", "expectedResult");
+ function verifyResult() {
+ shouldBe("listenersCalled", "expectedResult");
+ finishJSTest();
+ }
</script>

Powered by Google App Engine
This is Rietveld 408576698