[BlackBerry] requestAnimationFrame: Unscheduled display link frames need to be sent...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Aug 2012 00:07:56 +0000 (00:07 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Aug 2012 00:07:56 +0000 (00:07 +0000)
https://bugs.webkit.org/show_bug.cgi?id=94600

Patch by Andrew Lo <anlo@rim.com> on 2012-08-21
Reviewed by Rob Buis.

http://trac.webkit.org/changeset/116792 implements destroying
DisplayRefreshMonitor after the number of unscheduled frames exceeds 10.

Part of that change involves dispatching to the main thread whether
the frame was scheduled or not.

Make the corresponding change for the BlackBerry port that was made for
DisplayRefreshMonitorMac.cpp.

Tests covered by http://trac.webkit.org/changeset/116792

* platform/graphics/blackberry/DisplayRefreshMonitorBlackBerry.cpp:
(WebCore::DisplayRefreshMonitor::displayLinkFired):

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/blackberry/DisplayRefreshMonitorBlackBerry.cpp

index 0c61bfc..123f324 100644 (file)
@@ -1,3 +1,24 @@
+2012-08-21  Andrew Lo  <anlo@rim.com>
+
+        [BlackBerry] requestAnimationFrame: Unscheduled display link frames need to be sent to main thread
+        https://bugs.webkit.org/show_bug.cgi?id=94600
+
+        Reviewed by Rob Buis.
+
+        http://trac.webkit.org/changeset/116792 implements destroying
+        DisplayRefreshMonitor after the number of unscheduled frames exceeds 10.
+
+        Part of that change involves dispatching to the main thread whether
+        the frame was scheduled or not.
+
+        Make the corresponding change for the BlackBerry port that was made for
+        DisplayRefreshMonitorMac.cpp.
+
+        Tests covered by http://trac.webkit.org/changeset/116792
+
+        * platform/graphics/blackberry/DisplayRefreshMonitorBlackBerry.cpp:
+        (WebCore::DisplayRefreshMonitor::displayLinkFired):
+
 2012-08-21  Brian Salomon  <bsalomon@google.com>
 
         [Chromium/Skia] Filters should flush the SkCanvas
index 19d2673..b85afc3 100644 (file)
@@ -77,7 +77,7 @@ void DisplayRefreshMonitor::displayLinkFired()
     if (!m_mutex.tryLock())
         return;
 
-    if (!m_scheduled || !m_previousFrameDone) {
+    if (!m_previousFrameDone) {
         m_mutex.unlock();
         return;
     }