[SPInvalidation] Call layoutView->setNeedsPaintPropertyUpdate in FrameView::setNeedsPaintPropertyUp…

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 1301 matching lines @@
   RELEASE_ASSERT(!layoutViewItem().isNull());
   if (!RuntimeEnabledFeatures::rootLayerScrollingEnabled())
     invalidatePaintOfScrollControlsIfNeeded(paintInvalidationState);
 
   if (m_frame->selection().isCaretBoundsDirty())
     m_frame->selection().invalidateCaretRect();
 }
 
 void FrameView::setNeedsPaintPropertyUpdate() {
   m_needsPaintPropertyUpdate = true;
+  if (RuntimeEnabledFeatures::rootLayerScrollingEnabled()) {

[chrishtr, 2016/12/27 18:05:46]

Why wouldn't the cases with RLS enabled be handled via regular overflow
scroll invalidation paths?

[Xianzhu, 2016/12/27 18:57:32]

For scroll clip changes, the regular path is in LayoutBox::sizeChanged() which
doesn't apply to LayoutView whose overflow clip rect is derived from
FrameView::visibleContentRect() instead of LayoutBox::frameRect()
(https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/layout/La...).

+    if (auto* layoutView = this->layoutView()) {
+      layoutView->setNeedsPaintPropertyUpdate();
+      return;
+    }
+  }
   if (LayoutObject* owner = frame().ownerLayoutObject())
     owner->setNeedsPaintPropertyUpdate();
 }
 
 void FrameView::setSubtreeNeedsPaintPropertyUpdate() {
   setNeedsPaintPropertyUpdate();
   layoutView()->setSubtreeNeedsPaintPropertyUpdate();
 }
 
 IntRect FrameView::computeVisibleArea() {
@@ skipping 3561 matching lines @@
        reason < MainThreadScrollingReason::kMainThreadScrollingReasonCount;
        ++reason) {
     if (m_mainThreadScrollingReasonsCounter[reason] > 0) {
       reasons |= 1 << (reason - 1);
     }
   }
   return reasons;
 }
 
 }  // namespace blink