[BlackBerry] Don't suspend accel comp when backgrounding a page.

https://bugs.webkit.org/show_bug.cgi?id=80369
RIM PR: 137609

Reviewed by Adam Treat.

Just because the page is in the background doesn't mean
it's invisible, doesn't mean it won't have to be redrawn
at some point - on rotation, for instance.

We still suspend accelerated compositing on setVisible(false).

* Api/WebPage.cpp:
(BlackBerry::WebKit::WebPage::notifyPageBackground):
(BlackBerry::WebKit::WebPage::notifyPageForeground):

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

diff --git a/Source/WebKit/blackberry/Api/WebPage.cpp b/Source/WebKit/blackberry/Api/WebPage.cpp
index 5934391..d4b1a6e 100644 (file)
--- a/Source/WebKit/blackberry/Api/WebPage.cpp
+++ b/Source/WebKit/blackberry/Api/WebPage.cpp
@@ -4908,20 +4908,12 @@ void WebPage::notifyPageResume()
 
 void WebPage::notifyPageBackground()
 {
-#if USE(ACCELERATED_COMPOSITING)
-    d->suspendRootLayerCommit();
-#endif
-
     FOR_EACH_PLUGINVIEW(d->m_pluginViews)
         (*it)->handleBackgroundEvent();
 }
 
 void WebPage::notifyPageForeground()
 {
-#if USE(ACCELERATED_COMPOSITING)
-    d->resumeRootLayerCommit();
-#endif
-
     FOR_EACH_PLUGINVIEW(d->m_pluginViews)
         (*it)->handleForegroundEvent();
 }

diff --git a/Source/WebKit/blackberry/ChangeLog b/Source/WebKit/blackberry/ChangeLog
index ee8766c..c83be55 100644 (file)
--- a/Source/WebKit/blackberry/ChangeLog
+++ b/Source/WebKit/blackberry/ChangeLog
@@ -1,3 +1,21 @@
+2012-03-06  Jakob Petsovits  <jpetsovits@rim.com>
+
+        [BlackBerry] Don't suspend accel comp when backgrounding a page.
+        https://bugs.webkit.org/show_bug.cgi?id=80369
+        RIM PR: 137609
+
+        Reviewed by Adam Treat.
+
+        Just because the page is in the background doesn't mean
+        it's invisible, doesn't mean it won't have to be redrawn
+        at some point - on rotation, for instance.
+
+        We still suspend accelerated compositing on setVisible(false).
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPage::notifyPageBackground):
+        (BlackBerry::WebKit::WebPage::notifyPageForeground):
+
 2012-03-06  Jonathan Dong  <jonathan.dong@torchmobile.com.cn>
 
         [BlackBerry] function rename issue after MediaPlayerPrivateBlackBerry.[cpp\h] upstream