Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(228)

Unified Diff: third_party/WebKit/Source/core/frame/FrameView.cpp

Issue 2432933005: Revert "Log to UMA why we would load a third-party frame" (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 35d4fbbbbf42bc5802b849f3a6507da5d2daf093..419d75ec172dbeecae388a6c79699fff76fb6e8e 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -4308,28 +4308,11 @@ void FrameView::updateViewportIntersectionIfNeeded() {
m_viewportIntersectionValid = true;
FrameView* parent = parentFrameView();
if (!parent) {
- HTMLFrameOwnerElement* element = frame().deprecatedLocalOwner();
- if (!element)
- frame().document()->maybeRecordLoadReason(WouldLoadOutOfProcess);
- // Having no layout object means the frame is not drawn.
- else if (!element->layoutObject())
- frame().document()->maybeRecordLoadReason(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()->maybeRecordLoadReason(WouldLoadZeroByZero);
- else if (frameRect().maxY() < 0)
- frame().document()->maybeRecordLoadReason(WouldLoadAbove);
- else if (frameRect().maxX() < 0)
- frame().document()->maybeRecordLoadReason(WouldLoadLeft);
- }
-
// If our parent is hidden, then we are too.
if (parent->m_viewportIntersection.isEmpty()) {
m_viewportIntersection = parent->m_viewportIntersection;
@@ -4349,9 +4332,6 @@ void FrameView::updateViewportIntersectionIfNeeded() {
// content while scrolling.
IntRect viewport = parent->m_viewportIntersection;
m_viewportIntersection.intersect(viewport);
-
- if (parentLoaded && !m_viewportIntersection.isEmpty())
- frame().document()->maybeRecordLoadReason(WouldLoadVisible);
}
void FrameView::updateViewportIntersectionsForSubtree(
@@ -4437,6 +4417,8 @@ void FrameView::notifyRenderThrottlingObservers() {
updateThrottlingStatus();
+ frame().document()->onVisibilityMaybeChanged(!m_hiddenForThrottling);
+
bool becameThrottled = !wasThrottled && canThrottleRendering();
bool becameUnthrottled = wasThrottled && !canThrottleRendering();
ScrollingCoordinator* scrollingCoordinator = this->scrollingCoordinator();
« no previous file with comments | « third_party/WebKit/Source/core/dom/Document.cpp ('k') | third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698