Blink changes to record interest rects for http://w3c.github.io/frame-timing/

Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 68 matching lines @@
 #include "core/page/DragData.h"
 #include "core/page/DragSession.h"
 #include "core/page/EventHandler.h"
 #include "core/page/FocusController.h"
 #include "core/page/FrameTree.h"
 #include "core/page/Page.h"
 #include "core/page/PagePopupClient.h"
 #include "core/page/PointerLockController.h"
 #include "core/page/ScopedPageLoadDeferrer.h"
 #include "core/page/TouchDisambiguation.h"
+#include "core/paint/DeprecatedPaintLayer.h"
+#include "core/timing/DOMWindowPerformance.h"
+#include "core/timing/Performance.h"
 #include "modules/accessibility/AXObject.h"
 #include "modules/accessibility/AXObjectCacheImpl.h"
 #include "modules/credentialmanager/CredentialManagerClient.h"
 #include "modules/encryptedmedia/MediaKeysController.h"
 #include "modules/storage/StorageNamespaceController.h"
 #include "platform/ContextMenu.h"
 #include "platform/ContextMenuItem.h"
 #include "platform/Cursor.h"
 #include "platform/KeyboardCodes.h"
 #include "platform/Logging.h"
@@ skipping 19 matching lines @@
 #include "public/platform/WebFloatPoint.h"
 #include "public/platform/WebGestureCurve.h"
 #include "public/platform/WebImage.h"
 #include "public/platform/WebLayerTreeView.h"
 #include "public/platform/WebURLRequest.h"
 #include "public/platform/WebVector.h"
 #include "public/web/WebAXObject.h"
 #include "public/web/WebActiveWheelFlingParameters.h"
 #include "public/web/WebAutofillClient.h"
 #include "public/web/WebBeginFrameArgs.h"
+#include "public/web/WebFrame.h"
 #include "public/web/WebFrameClient.h"
 #include "public/web/WebHitTestResult.h"
 #include "public/web/WebInputElement.h"
 #include "public/web/WebMediaPlayerAction.h"
 #include "public/web/WebNode.h"
 #include "public/web/WebPlugin.h"
 #include "public/web/WebPluginAction.h"
 #include "public/web/WebRange.h"
 #include "public/web/WebSelection.h"
 #include "public/web/WebTextInputInfo.h"
@@ skipping 1768 matching lines @@
     if (m_continuousPaintingEnabled) {
         ContinuousPainter::setNeedsDisplayRecursive(m_rootGraphicsLayer, m_pageOverlays.get());
         m_client->scheduleAnimation();
     }
 }
 
 void WebViewImpl::setNeedsLayoutAndFullPaintInvalidation()
 {
     if (!mainFrameImpl() || !mainFrameImpl()->frame() || !mainFrameImpl()->frame()->view())
         return;
-
     FrameView* view = mainFrameImpl()->frame()->view();
     view->setNeedsLayout();
     view->setNeedsFullPaintInvalidation();
 }
 
+static void pushFrameTimingRequestRectsToGraphicsLayer(Page* page)

[chrishtr, 2015/04/17 23:59:57]

Are you intending to collect here the sizes of all of the graphics layers? cc
already knows the sizes of them, so why do you need to do this? All I can see
being necessary is to send the frame id for the graphics layer, right?

+{
+    typedef WTF::HashMap<const GraphicsLayer*,

[chrishtr, 2015/04/30 01:07:33]

Move this function to FrameView, which will avoid the for loop over Frame.

[MikeB, 2015/05/07 23:09:48]

Done.

+        std::vector<std::pair<int64_t, WebRect>>>
+        GraphicsLayerFrameTimingRequests;
+
+    GraphicsLayerFrameTimingRequests glRequests;

[chrishtr, 2015/04/30 01:07:33]

s/glRequests/graphicsLayerTimingRequests/

[MikeB, 2015/05/07 23:09:47]

Done.

+
+    for (Frame* frame = page ? page->mainFrame() : 0; frame;
+        frame = frame->tree().traverseNext()) {
+
+        // Need a LocalFrame to get the GraphicsLayer to use.
+        if (!frame->isLocalFrame())
+            continue;
+
+        LocalFrame* localframe = toLocalFrame(frame);

[chrishtr, 2015/04/30 01:07:33]

s/localframe/localFrame/

[MikeB, 2015/05/07 23:09:47]

Done.

+        Document* document = localframe->document();
+        HTMLFrameOwnerElement* ownerElement = document->ownerElement();
+
+        const GraphicsLayer* graphicsLayer = nullptr;
+
+        // Find frame's rect in graphics layer space
+        LayoutRect rect = localframe->contentRenderer()->viewRect();
+        DeprecatedPaintLayer::mapRectToPaintInvalidationBacking(

[chrishtr, 2015/04/30 01:07:33]

I think you should be able to simply write:

DeprecatedPaintLayer::mapRectToPaintInvalidationBacking(localFrame->contentRenderer(),
 localFrame>enclosingLayerForPaintInvalidationCrossingFrameBoundaries(),
localFrame->viewRect());

and

GraphicsLayer* graphicsLayer =
localFrame>enclosingLayerForPaintInvalidationCrossingFrameBoundaries()->graphicsLayerBacking()


and avoid lines 1947-1970

[MikeB, 2015/05/07 23:09:47]

enclosingLayerForPaintInvalidationCrossingFrameBoundaries is not a property on
blink::LocalFrame. It only lives on DeprecatedPaintLayer, so I have to keep the
->enclosingLayer()

LocalFrame also doesn't have a viewRect() property. Does this sound about right?

LayoutRect rect = localFrame->contentRenderer()->viewRect();

DeprecatedPaintLayer::mapRectToPaintInvalidationBacking(
   localFrame->contentRenderer(),
   localFrame->contentRenderer()->containerForPaintInvalidation(),
   rect);

const GraphicsLayer* graphicsLayer =
localFrame->contentRenderer()->enclosingLayer()->enclosingLayerForPaintInvalidationCrossingFrameBoundaries()->graphicsLayerBacking();

+            localframe->contentRenderer(),
+            localframe->contentRenderer()->containerForPaintInvalidation(),
+            rect);
+
+        if (document->layoutView()->enclosingLayer()->compositingState()
+            == PaintsIntoOwnBacking || !ownerElement) {
+            DeprecatedPaintLayer* layer = document->layoutView()
+                ->enclosingLayer()
+                ->enclosingLayerForPaintInvalidationCrossingFrameBoundaries();
+            graphicsLayer = layer ? layer->graphicsLayerBacking() : nullptr;
+        } else if (ownerElement->layoutObject()) {
+            DeprecatedPaintLayer* layer = ownerElement->layoutObject()
+                ->enclosingLayer()
+                ->enclosingLayerForPaintInvalidationCrossingFrameBoundaries();
+            graphicsLayer = layer ? layer->graphicsLayerBacking() : nullptr;
+            DeprecatedPaintLayer::mapRectToPaintInvalidationBacking(
+                ownerElement->layoutObject(),
+                ownerElement->layoutObject()->containerForPaintInvalidation(),
+                rect);
+        }
+        if (!graphicsLayer)
+            continue;
+
+        GraphicsLayerFrameTimingRequests::iterator requestIterator = glRequests.find(graphicsLayer);
+        std::vector<std::pair<int64_t, WebRect>> *graphicsLayerTimingRects;
+        if (requestIterator == glRequests.end()) {
+            graphicsLayerTimingRects = &glRequests.add(graphicsLayer,
+                std::vector<std::pair<int64_t, WebRect>>()).storedValue->value;
+        } else {
+            graphicsLayerTimingRects = &requestIterator->value;
+        }
+        graphicsLayerTimingRects->push_back(std::make_pair(frame->frameID(), enclosingIntRect(rect)));
+    }
+
+    for (GraphicsLayerFrameTimingRequests::const_iterator iter = glRequests.begin(); iter != glRequests.end(); ++iter) {

[chrishtr, 2015/04/30 01:07:33]

Change WebLayer to have a method addFrameTimingRequest instead of
setFrameTimingRequests so that this code is compatible with moving to FrameView.

[MikeB, 2015/05/07 23:09:48]

Not yet done. Will see about making this change once code is moved to FrameView.

+        const GraphicsLayer* graphicsLayer = iter->key;
+        graphicsLayer->platformLayer()->setFrameTimingRequests(iter->value);
+    }
+}
+
 void WebViewImpl::layout()
 {
     TRACE_EVENT0("blink", "WebViewImpl::layout");
     if (!mainFrameImpl())
         return;
 
     PageWidgetDelegate::layout(*m_page, *mainFrameImpl()->frame());
     updateLayerTreeBackgroundColor();
 
     for (size_t i = 0; i < m_linkHighlights.size(); ++i)
         m_linkHighlights[i]->updateGeometry();
+
+    if (RuntimeEnabledFeatures::frameTimingSupportEnabled())
+        pushFrameTimingRequestRectsToGraphicsLayer(m_page.get());

[chrishtr, 2015/04/30 01:07:33]

I think this should go in FrameView::updateLayoutAndStyleForPainting instead.
ScrollingCoordinator fulfills a similar function to plumb touch event rects to
graphics layers.

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

You should also use a dirty bit to avoid this work, similar again to what
ScrollingCoordinator does.

[MikeB, 2015/05/07 23:09:48]

Done, except for the dirty bit.

 }
 
 void WebViewImpl::paint(WebCanvas* canvas, const WebRect& rect)
 {
     // This should only be used when compositing is not being used for this
     // WebView, and it is painting into the recording of its parent.
     ASSERT(!isAcceleratedCompositingActive());
 
     double paintStart = currentTime();
     PageWidgetDelegate::paint(*m_page, pageOverlays(), canvas, rect, *m_page->deprecatedLocalMainFrame());
@@ skipping 2360 matching lines @@
 
     if (pageScaleDelta != 1)
         m_doubleTapZoomPending = false;
 
     frameView->setElasticOverscroll(elasticOverscrollDelta + frameView->elasticOverscroll());
 
     updateMainFrameScrollPosition(frameView->scrollableArea()->scrollPositionDouble() +
         DoubleSize(outerViewportDelta.width, outerViewportDelta.height), /* programmaticScroll */ false);
 }
 
+void WebViewImpl::recordFrameTimingEvent(FrameTimingEventType eventType, int64_t FrameId, const WebVector<std::pair<int, double>>& events)
+{
+    Frame* frame = m_page ? m_page->mainFrame() : 0;
+
+    while (frame && frame->frameID() != FrameId) {
+        frame = frame->tree().traverseNext();
+    }
+
+    if (!frame || !frame->domWindow() || !frame->domWindow()->document())
+        return; // Can't find frame, it may have been cleaned up from the DOM.
+
+    blink::DOMWindow* domWindow = frame->domWindow();
+    blink::Performance* performance = DOMWindowPerformance::performance(*domWindow);
+    for (size_t i = 0; i < events.size(); ++i) {
+        if (eventType == CompositeEvent)
+            performance->addCompositeTiming(domWindow->document(), events[i].first, events[i].second);
+        else if (eventType == RenderEvent)
+            performance->addRenderTiming(domWindow->document(), events[i].first, events[i].second);
+    }
+}
+
 void WebViewImpl::updateLayerTreeViewport()
 {
     if (!page() || !m_layerTreeView)
         return;
 
     m_layerTreeView->setPageScaleFactorAndLimits(pageScaleFactor(), minimumPageScaleFactor(), maximumPageScaleFactor());
 }
 
 void WebViewImpl::updateLayerTreeBackgroundColor()
 {
@@ skipping 134 matching lines @@
     return mainFrameImpl()->frameView()->layoutSize().width() == m_size.width
         || (constraints.minimumScale == constraints.maximumScale && constraints.minimumScale != -1);
 }
 
 void WebViewImpl::forceNextWebGLContextCreationToFail()
 {
     WebGLRenderingContext::forceNextWebGLContextCreationToFail();
 }
 
 } // namespace blink