Changeset 147032 in webkit

Timestamp:

Mar 27, 2013 5:18:06 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[BlackBerry] Show compositing on the Inspector timeline properly
​https://bugs.webkit.org/show_bug.cgi?id=113416

Patch by Eli Fidler <​efidler@blackberry.com> on 2013-03-27
Reviewed by Rob Buis.

The call to willComposite was too early and encompased the paint calls
as well. Moving it lower will no longer encompass the paints.

• Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::commitRootLayerIfNeeded):

Location:

trunk/Source/WebKit/blackberry

Files:

• Api/WebPage.cpp (modified) (2 diffs)
• ChangeLog (modified) (1 diff)

trunk/Source/WebKit/blackberry/Api/WebPage.cpp

@@ -5529 +5529 @@
     }
 
-    willComposite();
     m_needsCommit = false;
     // We get here either due to the commit timer, which would have called
@@ -5546 +5545 @@
         m_overlayLayer->platformLayer()->commitOnWebKitThread(scale);
 
+    willComposite();
     // Stash the visible content rect according to webkit thread
     // This is the rectangle used to layout fixed positioned elements,

trunk/Source/WebKit/blackberry/ChangeLog

@@ +1 @@
+2013-03-27  Eli Fidler <efidler@blackberry.com>
+
+        [BlackBerry] Show compositing on the Inspector timeline properly
+        https://bugs.webkit.org/show_bug.cgi?id=113416
+
+        Reviewed by Rob Buis.
+
+        The call to willComposite was too early and encompased the paint calls
+        as well.  Moving it lower will no longer encompass the paints.
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPagePrivate::commitRootLayerIfNeeded):
+
 2013-03-27  Arvid Nilsson  <anilsson@rim.com>