2011-01-10 Maciej Stachowiak <mjs@apple.com>
authormjs@apple.com <mjs@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Jan 2011 20:00:42 +0000 (20:00 +0000)
committermjs@apple.com <mjs@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Jan 2011 20:00:42 +0000 (20:00 +0000)
        Reviewed by Anders Carlsson.

        WebKitTestRunner uses crazy amounts of CPU, making layout tests slow and flaky
        https://bugs.webkit.org/show_bug.cgi?id=52160

        * WebKitTestRunner/mac/TestControllerMac.mm:
        (WTR::TestController::platformRunUntil): Instead of busy-looping, wait for the timeout.
        Anything that can make runUntil() conditions turn true will happen on a run loop cycle
        anyway.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@75401 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/WebKitTestRunner/mac/TestControllerMac.mm

index 36eff69f95678532141cd550699172f0ad09b115..874e892127a68b49a6038f3975c7ab72384762b6 100644 (file)
@@ -1,3 +1,15 @@
+2011-01-10  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Anders Carlsson.
+
+        WebKitTestRunner uses crazy amounts of CPU, making layout tests slow and flaky
+        https://bugs.webkit.org/show_bug.cgi?id=52160
+
+        * WebKitTestRunner/mac/TestControllerMac.mm:
+        (WTR::TestController::platformRunUntil): Instead of busy-looping, wait for the timeout.
+        Anything that can make runUntil() conditions turn true will happen on a run loop cycle
+        anyway.
+
 2011-01-10  Tony Chang  <tony@chromium.org>
 
         Reviewed by Mihai Parparita.
index fee007070569184d72ef0ed5ed46226f5949bbb2..6a4444fd473cd1126caa6dae1ff895814c48f8de 100644 (file)
@@ -52,8 +52,10 @@ void TestController::initializeTestPluginDirectory()
 void TestController::platformRunUntil(bool& done, double timeout)
 {
     CFAbsoluteTime end = CFAbsoluteTimeGetCurrent() + timeout;
+    CFDateRef endDate = CFDateCreate(0, end);
     while (!done && CFAbsoluteTimeGetCurrent() < end)
-        [[NSRunLoop currentRunLoop] runMode:NSDefaultRunLoopMode beforeDate:[NSDate distantPast]];
+        [[NSRunLoop currentRunLoop] runMode:NSDefaultRunLoopMode beforeDate:(NSDate *)endDate];
+    CFRelease(endDate);
 }
 
 void TestController::platformInitializeContext()