Title: [133477] trunk/Source/WebKit/blackberry
Revision
133477
Author
[email protected]
Date
2012-11-05 07:42:42 -0800 (Mon, 05 Nov 2012)

Log Message

[BlackBerry] Web Inspector: Add InspectorInstrumentaiton calls for timeline compositing.
https://bugs.webkit.org/show_bug.cgi?id=101213
RIM PR 178131

Reviewed by Rob Buis.

Internally Reviewed by Arvid Nilsson.

Added compositing events to the timeline for BlackBerry.

* Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::commitRootLayerIfNeeded):
(BlackBerry::WebKit::WebPagePrivate::willComposite):
(WebKit):
(BlackBerry::WebKit::WebPagePrivate::didComposite):
* Api/WebPage_p.h:
(WebPagePrivate):

Modified Paths

Diff

Modified: trunk/Source/WebKit/blackberry/Api/WebPage.cpp (133476 => 133477)


--- trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2012-11-05 15:29:44 UTC (rev 133476)
+++ trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2012-11-05 15:42:42 UTC (rev 133477)
@@ -5507,6 +5507,7 @@
         return false;
     }
 
+    willComposite();
     m_needsCommit = false;
     // We get here either due to the commit timer, which would have called
     // render if a one shot sync was needed. Or we get called from render
@@ -5539,6 +5540,7 @@
             contentsSizeForCompositing,
             drawsRootLayer));
 
+    didComposite();
     return true;
 }
 
@@ -6258,5 +6260,19 @@
     m_cachedHitTestContentPos = WebCore::IntPoint(-1, -1);
 }
 
+void WebPagePrivate::willComposite()
+{
+    if (!m_page->settings()->developerExtrasEnabled())
+        return;
+    InspectorInstrumentation::willComposite(m_page);
 }
+
+void WebPagePrivate::didComposite()
+{
+    if (!m_page->settings()->developerExtrasEnabled())
+        return;
+    InspectorInstrumentation::didComposite(m_page);
 }
+
+}
+}

Modified: trunk/Source/WebKit/blackberry/Api/WebPage_p.h (133476 => 133477)


--- trunk/Source/WebKit/blackberry/Api/WebPage_p.h	2012-11-05 15:29:44 UTC (rev 133476)
+++ trunk/Source/WebKit/blackberry/Api/WebPage_p.h	2012-11-05 15:42:42 UTC (rev 133477)
@@ -468,6 +468,9 @@
 
     WebCore::IntSize screenSize() const;
 
+    void willComposite();
+    void didComposite();
+
     WebPage* m_webPage;
     WebPageClient* m_client;
     WebCore::InspectorClientBlackBerry* m_inspectorClient;

Modified: trunk/Source/WebKit/blackberry/ChangeLog (133476 => 133477)


--- trunk/Source/WebKit/blackberry/ChangeLog	2012-11-05 15:29:44 UTC (rev 133476)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2012-11-05 15:42:42 UTC (rev 133477)
@@ -1,3 +1,23 @@
+2012-11-05  Konrad Piascik  <[email protected]>
+
+        [BlackBerry] Web Inspector: Add InspectorInstrumentaiton calls for timeline compositing.
+        https://bugs.webkit.org/show_bug.cgi?id=101213
+        RIM PR 178131
+
+        Reviewed by Rob Buis.
+
+        Internally Reviewed by Arvid Nilsson.
+
+        Added compositing events to the timeline for BlackBerry.
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPagePrivate::commitRootLayerIfNeeded):
+        (BlackBerry::WebKit::WebPagePrivate::willComposite):
+        (WebKit):
+        (BlackBerry::WebKit::WebPagePrivate::didComposite):
+        * Api/WebPage_p.h:
+        (WebPagePrivate):
+
 2012-11-03  Alexey Proskuryakov  <[email protected]>
 
         Get rid of USE(CFURLSTORAGESESSIONS)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to