Update paint properties on FrameView visibility changes

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 4409 matching lines @@
 }
 
 void FrameView::show() {
   if (!isSelfVisible()) {
     setSelfVisible(true);
     if (ScrollingCoordinator* scrollingCoordinator =
             this->scrollingCoordinator())
       scrollingCoordinator->frameViewVisibilityDidChange();
     setNeedsCompositingUpdate(layoutViewItem(), CompositingUpdateRebuildTree);
     updateParentScrollableAreaSet();
+    if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled()) {
+      // The existance of scrolling properties depends on visibility through
+      // isScrollable() so ensure properties are updated if visibility changes.
+      setNeedsPaintPropertyUpdate();
+    }
     if (isParentVisible()) {
       for (const auto& child : m_children)
         child->setParentVisible(true);
     }
   }
 
   Widget::show();
 }
 
 void FrameView::hide() {
   if (isSelfVisible()) {
     if (isParentVisible()) {
       for (const auto& child : m_children)
         child->setParentVisible(false);
     }
     setSelfVisible(false);
     if (ScrollingCoordinator* scrollingCoordinator =
             this->scrollingCoordinator())
       scrollingCoordinator->frameViewVisibilityDidChange();
     setNeedsCompositingUpdate(layoutViewItem(), CompositingUpdateRebuildTree);
     updateParentScrollableAreaSet();
+    if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled()) {
+      // The existance of scrolling properties depends on visibility through
+      // isScrollable() so ensure properties are updated if visibility changes.
+      setNeedsPaintPropertyUpdate();
+    }
   }
 
   Widget::hide();
 }
 
 int FrameView::viewportWidth() const {
   int viewportWidth = layoutSize(IncludeScrollbars).width();
   return adjustForAbsoluteZoom(viewportWidth, layoutView());
 }
 
@@ skipping 222 matching lines @@
   DCHECK(m_frame->isMainFrame());
   return m_initialViewportSize.width();
 }
 
 int FrameView::initialViewportHeight() const {
   DCHECK(m_frame->isMainFrame());
   return m_initialViewportSize.height();
 }
 
 }  // namespace blink