Deferred frame loading stats v2

third_party/WebKit/Source/core/frame/FrameView.cpp

Index: third_party/WebKit/Source/core/frame/FrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
index 5cfb85f3f5f5fddc20055ed245de877eeb10987f..61c939b25d90b92ea65c16a2f5d6eba41ea85af8 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -4311,11 +4311,31 @@ void FrameView::updateViewportIntersectionIfNeeded() {
   m_viewportIntersectionValid = true;
   FrameView* parent = parentFrameView();
   if (!parent) {
+    HTMLFrameOwnerElement* element = frame().deprecatedLocalOwner();
+    // Determining visibility uses the owner element's LayoutObject,

[dgrogan, 2016/10/18 22:11:45]

I used this comment without thinking about it, but it doesn't seem to be true.
We're using the FrameView's frameRect(), not the LayoutObject.

+    // which is not currently possible if the parent is
+    // remote.
+    if (!element)
+      frame().document()->wouldLoadBecause(WouldLoadOutOfProcess);
+    // Having no layout object means the frame is not drawn.
+    else if (!element->layoutObject())
+      frame().document()->wouldLoadBecause(WouldLoadDisplayNone);
     m_viewportIntersection = frameRect();
     return;
   }
   ASSERT(!parent->m_needsUpdateViewportIntersection);
 
+  bool parentLoaded = parent->frame().document()->wouldLoadReason() > Created;
+  // If the parent wasn't loaded, the children won't be either.
+  if (parentLoaded) {
+    if (frameRect().isEmpty())
+      frame().document()->wouldLoadBecause(WouldLoadZeroByZero);
+    else if (frameRect().maxY() < 0)
+      frame().document()->wouldLoadBecause(WouldLoadAbove);
+    else if (frameRect().maxX() < 0)
+      frame().document()->wouldLoadBecause(WouldLoadLeft);
+  }
+
   // If our parent is hidden, then we are too.
   if (parent->m_viewportIntersection.isEmpty()) {
     m_viewportIntersection = parent->m_viewportIntersection;
@@ -4335,6 +4355,9 @@ void FrameView::updateViewportIntersectionIfNeeded() {
   // content while scrolling.
   IntRect viewport = parent->m_viewportIntersection;
   m_viewportIntersection.intersect(viewport);
+
+  if (parentLoaded && !m_viewportIntersection.isEmpty())
+    frame().document()->wouldLoadBecause(WouldLoadVisible);
 }
 
 void FrameView::updateViewportIntersectionsForSubtree(
@@ -4420,8 +4443,6 @@ void FrameView::notifyRenderThrottlingObservers() {
 
   updateThrottlingStatus();
 
-  frame().document()->onVisibilityMaybeChanged(!m_hiddenForThrottling);
-
   bool becameThrottled = !wasThrottled && canThrottleRendering();
   bool becameUnthrottled = wasThrottled && !canThrottleRendering();
   ScrollingCoordinator* scrollingCoordinator = this->scrollingCoordinator();