2011-02-22 Kenneth Russell <kbr@google.com>
authorkbr@google.com <kbr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Feb 2011 03:23:09 +0000 (03:23 +0000)
committerkbr@google.com <kbr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Feb 2011 03:23:09 +0000 (03:23 +0000)
        Reviewed by James Robinson.

        Many tests started timing out on the chrome mac canaries around r79035
        https://bugs.webkit.org/show_bug.cgi?id=54885

        Revert the code changes, but not the Chromium DEPS roll, from
        r79038 to see whether this clears up the timing-out tests. Was not
        able to reproduce the timeouts locally with a Release mode build
        of DRT (on Snow Leopard).

        * src/WebKit.cpp:
        (WebKit::initialize):
2011-02-22  Kenneth Russell  <kbr@google.com>

        Reviewed by James Robinson.

        Many tests started timing out on the chrome mac canaries around r79035
        https://bugs.webkit.org/show_bug.cgi?id=54885

        Revert the code changes, but not the Chromium DEPS roll, from
        r79038 to see whether this clears up the timing-out tests. Was not
        able to reproduce the timeouts locally with a Release mode build
        of DRT (on Snow Leopard).

        * DumpRenderTree/chromium/WebViewHost.cpp:
        (WebViewHost::reset):

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

Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/src/WebKit.cpp
Tools/ChangeLog
Tools/DumpRenderTree/chromium/WebViewHost.cpp

index 760bdac..fe7191f 100644 (file)
@@ -1,3 +1,18 @@
+2011-02-22  Kenneth Russell  <kbr@google.com>
+
+        Reviewed by James Robinson.
+
+        Many tests started timing out on the chrome mac canaries around r79035
+        https://bugs.webkit.org/show_bug.cgi?id=54885
+
+        Revert the code changes, but not the Chromium DEPS roll, from
+        r79038 to see whether this clears up the timing-out tests. Was not
+        able to reproduce the timeouts locally with a Release mode build
+        of DRT (on Snow Leopard).
+
+        * src/WebKit.cpp:
+        (WebKit::initialize):
+
 2011-02-21  Nico Weber  <thakis@chromium.org>
 
         Reviewed by James Robinson.
index d130dd5..471fd39 100644 (file)
@@ -66,6 +66,15 @@ void initialize(WebKitClient* webKitClient)
     WTF::initializeMainThread();
     WTF::AtomicString::init();
 
+    // Chromium sets the minimum interval timeout to 4ms, overriding the
+    // default of 10ms.  We'd like to go lower, however there are poorly
+    // coded websites out there which do create CPU-spinning loops.  Using
+    // 4ms prevents the CPU from spinning too busily and provides a balance
+    // between CPU spinning and the smallest possible interval timer.
+    // FIXME: remove this call once Chromium has been changed to set
+    // this for all pages using the new WebSettings API.
+    WebCore::Settings::setDefaultMinDOMTimerInterval(0.004);
+
     // There are some code paths (for example, running WebKit in the browser
     // process and calling into LocalStorage before anything else) where the
     // UTF8 string encoding tables are used on a background thread before
index 449cf00..baba6e5 100644 (file)
@@ -1,3 +1,18 @@
+2011-02-22  Kenneth Russell  <kbr@google.com>
+
+        Reviewed by James Robinson.
+
+        Many tests started timing out on the chrome mac canaries around r79035
+        https://bugs.webkit.org/show_bug.cgi?id=54885
+
+        Revert the code changes, but not the Chromium DEPS roll, from
+        r79038 to see whether this clears up the timing-out tests. Was not
+        able to reproduce the timeouts locally with a Release mode build
+        of DRT (on Snow Leopard).
+
+        * DumpRenderTree/chromium/WebViewHost.cpp:
+        (WebViewHost::reset):
+
 2011-02-22  Dirk Pranke  <dpranke@chromium.org>
 
         Reviewed by James Robinson.
index a0ed6da..5ba0dcb 100644 (file)
@@ -1210,7 +1210,8 @@ void WebViewHost::reset()
 
     if (m_webWidget) {
         webView()->mainFrame()->setName(WebString());
-        webView()->settings()->setMinimumTimerInterval(webkit_support::GetForegroundTabTimerInterval());
+        // FIXME: change to use webkit_support::ForegroundTabTimerInterval once that lands.
+        webView()->settings()->setMinimumTimerInterval(0.004);
     }
 }