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

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

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

Powered by Google App Engine
This is Rietveld 408576698