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

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 be913bda96355836de374231f5630bf74456b949..05440d80bb55059c5edf195a95c89b0d8f214b8c 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -1319,6 +1319,12 @@ void FrameView::invalidatePaintIfNeeded(
 
 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();
 }