Title: [156486] trunk/LayoutTests
Revision
156486
Author
[email protected]
Date
2013-09-26 12:14:28 -0700 (Thu, 26 Sep 2013)

Log Message

fast/css/font-face-data-uri.html is failing
https://bugs.webkit.org/show_bug.cgi?id=82744

Reviewed by Dan Bernstein.

* fast/css/font-face-data-uri.html: There is no need to keep retrying, we can just
wait for onload - as long as the font load starts early enough.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (156485 => 156486)


--- trunk/LayoutTests/ChangeLog	2013-09-26 19:09:04 UTC (rev 156485)
+++ trunk/LayoutTests/ChangeLog	2013-09-26 19:14:28 UTC (rev 156486)
@@ -1,3 +1,13 @@
+2013-09-26  Alexey Proskuryakov  <[email protected]>
+
+        fast/css/font-face-data-uri.html is failing
+        https://bugs.webkit.org/show_bug.cgi?id=82744
+
+        Reviewed by Dan Bernstein.
+
+        * fast/css/font-face-data-uri.html: There is no need to keep retrying, we can just
+        wait for onload - as long as the font load starts early enough.
+
 2013-09-26  Commit Queue  <[email protected]>
 
         Unreviewed, rolling out r156451.

Modified: trunk/LayoutTests/fast/css/font-face-data-uri.html (156485 => 156486)


--- trunk/LayoutTests/fast/css/font-face-data-uri.html	2013-09-26 19:09:04 UTC (rev 156485)
+++ trunk/LayoutTests/fast/css/font-face-data-uri.html	2013-09-26 19:14:28 UTC (rev 156486)
@@ -17,7 +17,7 @@
 </style>
 <script src=""
 </head>
-<body _onload_="warmUpAndTest()">
+<body _onload_="testMain()">
 <div id="description"></div>
 <div id="console"></div>
 <hr/>
@@ -40,21 +40,9 @@
     finishJSTest();
 }
 
-var trialCount = 0;
+// Force layout, so that fonts begin to load before the document finishes loading, and thus delay the load event.
+document.body.offsetTop;
 
-function warmUpAndTest()
-{
-    document.body.offsetTop; // Force layout.
-    if (trialCount > 200
-        || document.getElementById("ref0").offsetWidth == document.getElementById("test0").offsetWidth
-        || document.getElementById("ref1").offsetWidth == document.getElementById("test1").offsetWidth) {
-        // Font has been loaded (or the wait limit has been exceeded). Now run the test.
-        testMain();
-        return;
-    }
-    trialCount++;
-    window.setTimeout(warmUpAndTest, 10);
-}
 </script>
 <script src=""
 </body>
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to