Get rid of obsolete FrameView::m_shouldUpdateViewportIntersection.

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

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 157 matching lines @@
       m_baseBackgroundColor(Color::white),
       m_mediaType(MediaTypeNames::screen),
       m_safeToPropagateScrollToParent(true),
       m_scrollCorner(nullptr),
       m_stickyPositionObjectCount(0),
       m_inputEventsScaleFactorForEmulation(1),
       m_layoutSizeFixedToFrameSize(true),
       m_didScrollTimer(this, &FrameView::didScrollTimerFired),
       m_browserControlsViewportAdjustment(0),
       m_needsUpdateWidgetGeometries(false),
-      m_needsUpdateViewportIntersection(true),
 #if ENABLE(ASSERT)
       m_hasBeenDisposed(false),
 #endif
       m_horizontalScrollbarMode(ScrollbarAuto),
       m_verticalScrollbarMode(ScrollbarAuto),
       m_horizontalScrollbarLock(false),
       m_verticalScrollbarLock(false),
       m_scrollbarsSuppressed(false),
       m_inUpdateScrollbars(false),
       m_frameTimingRequestsDirty(true),
@@ skipping 3388 matching lines @@
   m_animatingScrollableAreas->remove(scrollableArea);
 }
 
 void FrameView::setParent(Widget* parentView) {
   if (parentView == parent())
     return;
 
   Widget::setParent(parentView);
 
   updateParentScrollableAreaSet();
-  setNeedsUpdateViewportIntersection();
   setupRenderThrottling();
 
   if (parentFrameView())
     m_subtreeThrottled = parentFrameView()->canThrottleRendering();
 }
 
 void FrameView::removeChild(Widget* child) {
   ASSERT(child->parent() == this);
 
   if (child->isFrameView())
@@ skipping 30 matching lines @@
   if (!page || m_frame->eventHandler().isMousePositionUnknown())
     return;
   page->chromeClient().setCursor(cursor, m_frame);
 }
 
 void FrameView::frameRectsChanged() {
   TRACE_EVENT0("blink", "FrameView::frameRectsChanged");
   if (layoutSizeFixedToFrameSize())
     setLayoutSizeInternal(frameRect().size());
 
-  setNeedsUpdateViewportIntersection();
   if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled()) {
     // The overflow clip property depends on the frame rect.
     setNeedsPaintPropertyUpdate();
   }
 
   for (const auto& child : m_children)
     child->frameRectsChanged();
 }
 
 void FrameView::setLayoutSizeInternal(const IntSize& size) {
@@ skipping 894 matching lines @@
   // so we don't want to include them.
   if (layoutObject.isText())
     return;
 
   layoutObject.addAnnotatedRegions(regions);
   for (LayoutObject* curr = layoutObject.slowFirstChild(); curr;
        curr = curr->nextSibling())
     collectAnnotatedRegions(*curr, regions);
 }
 
-void FrameView::setNeedsUpdateViewportIntersection() {
-  for (FrameView* parent = parentFrameView(); parent;
-       parent = parent->parentFrameView())
-    parent->m_needsUpdateViewportIntersectionInSubtree = true;
-}
-
 void FrameView::updateViewportIntersectionsForSubtree(
     DocumentLifecycle::LifecycleState targetState) {
   // TODO(dcheng): Since widget tree updates are deferred, FrameViews might
   // still be in the widget hierarchy even though the associated Document is
   // already detached. Investigate if this check and a similar check in
   // lifecycle updates are still needed when there are no more deferred widget
   // updates: https://crbug.com/561683
   if (!frame().document()->isActive())
     return;
 
   // Notify javascript IntersectionObservers
   if (targetState == DocumentLifecycle::PaintClean &&
       frame().document()->intersectionObserverController())
     frame()
         .document()
         ->intersectionObserverController()
         ->computeTrackedIntersectionObservations();
 
-  if (!m_needsUpdateViewportIntersectionInSubtree)
-    return;
-  m_needsUpdateViewportIntersectionInSubtree = false;
-
   for (Frame* child = m_frame->tree().firstChild(); child;
        child = child->tree().nextSibling()) {
     if (!child->isLocalFrame())
       continue;
     if (FrameView* view = toLocalFrame(child)->view())
       view->updateViewportIntersectionsForSubtree(targetState);
   }
 }
 
 void FrameView::updateRenderThrottlingStatusForTesting() {
@@ skipping 325 matching lines @@
        reason < MainThreadScrollingReason::kMainThreadScrollingReasonCount;
        ++reason) {
     if (m_mainThreadScrollingReasonsCounter[reason] > 0) {
       reasons |= 1 << (reason - 1);
     }
   }
   return reasons;
 }
 
 }  // namespace blink