Title: [132461] trunk/LayoutTests
Revision
132461
Author
[email protected]
Date
2012-10-25 02:15:29 -0700 (Thu, 25 Oct 2012)

Log Message

Layout Test fast/media/w3c/test_media_queries.html is slow on Chromium debug bots after r132352
https://bugs.webkit.org/show_bug.cgi?id=100272

Patch by Alexander Shalamov <[email protected]> on 2012-10-25
Reviewed by Kenneth Rohde Christiansen.

Test if resolution media query is supported before identifying dpi of the media.

* fast/media/w3c/test_media_queries.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (132460 => 132461)


--- trunk/LayoutTests/ChangeLog	2012-10-25 08:48:46 UTC (rev 132460)
+++ trunk/LayoutTests/ChangeLog	2012-10-25 09:15:29 UTC (rev 132461)
@@ -1,3 +1,14 @@
+2012-10-25  Alexander Shalamov  <[email protected]>
+
+        Layout Test fast/media/w3c/test_media_queries.html is slow on Chromium debug bots after r132352
+        https://bugs.webkit.org/show_bug.cgi?id=100272
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        Test if resolution media query is supported before identifying dpi of the media.
+
+        * fast/media/w3c/test_media_queries.html:
+
 2012-10-25  Nandor Huszka  <[email protected]>
 
         [Qt] Unreviewed gardening.

Modified: trunk/LayoutTests/fast/media/w3c/test_media_queries.html (132460 => 132461)


--- trunk/LayoutTests/fast/media/w3c/test_media_queries.html	2012-10-25 08:48:46 UTC (rev 132460)
+++ trunk/LayoutTests/fast/media/w3c/test_media_queries.html	2012-10-25 09:15:29 UTC (rev 132461)
@@ -353,14 +353,21 @@
 
     // Find the resolution using max-resolution
     var resolution = 0;
-    do {
-      ++resolution;
-      if (resolution > 10000) {
-        break;
-      }
-    } while (!query_applies("(max-resolution: " + resolution + "dpi)"));
+    var max_resolution = 1000;
+
+    if (query_applies("(resolution)")) {
+        do {
+          ++resolution;
+          if (resolution > max_resolution) {
+            break;
+          }
+        } while (!query_applies("(max-resolution: " + resolution + "dpi)"));
+    } else {
+        resolution = max_resolution + 1;
+    }
+
     test(function() {
-      assert_false_wrapper(10000 < resolution);
+      assert_false_wrapper(max_resolution < resolution);
     }, "find_resolution");
 
     // resolution should now be Math.ceil() of the actual resolution.
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to