Changeset 152202 in webkit

Timestamp:

Jun 28, 2013 6:45:04 PM (11 years ago)

Author:

barraclough@apple.com

Message:

PageThrottler::shouldThrottleTimers is wrong.
​https://bugs.webkit.org/show_bug.cgi?id=118210

Reviewed by Anders Carlson.

m_throttleState == PageNotThrottledState

-> page NOT throttled, so should NOT throttle timers

m_throttleState != PageNotThrottledState

-> page NOT NOT throttled, so should throttle timers

This needs cleaning up, but for now, 1 character fix.

• page/PageThrottler.h:

(WebCore::PageThrottler::shouldThrottleTimers):

-> !=

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/PageThrottler.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-06-28  Gavin Barraclough  <barraclough@apple.com>
+
+        PageThrottler::shouldThrottleTimers is wrong.
+        https://bugs.webkit.org/show_bug.cgi?id=118210
+
+        Reviewed by Anders Carlson.
+
+        m_throttleState == PageNotThrottledState
+            -> page NOT throttled, so should NOT throttle timers
+        m_throttleState != PageNotThrottledState
+            -> page NOT NOT throttled, so should throttle timers
+
+        This needs cleaning up, but for now, 1 character fix.
+
+        * page/PageThrottler.h:
+        (WebCore::PageThrottler::shouldThrottleTimers):
+            == -> !=
+
 2013-06-28  Ryosuke Niwa  <rniwa@webkit.org>
 

trunk/Source/WebCore/page/PageThrottler.h

@@ -45 +45 @@
 
     bool shouldThrottleAnimations() const { return m_throttleState != PageNotThrottledState; }
-    bool shouldThrottleTimers() const { return m_throttleState == PageNotThrottledState; }
+    bool shouldThrottleTimers() const { return m_throttleState != PageNotThrottledState; }
 
     void setThrottled(bool);