[SPInvalidation] Update paint properties when full screen status changes

third_party/WebKit/Source/core/dom/Fullscreen.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 16 matching lines @@
  *
  */
 
 #include "core/dom/Fullscreen.h"
 
 #include "bindings/core/v8/ConditionalFeatures.h"
 #include "core/dom/Document.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/dom/StyleEngine.h"
 #include "core/events/Event.h"
+#include "core/frame/FrameView.h"
 #include "core/frame/HostsUsingFeatures.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
 #include "core/frame/UseCounter.h"
 #include "core/html/HTMLIFrameElement.h"
 #include "core/input/EventHandler.h"
 #include "core/inspector/ConsoleMessage.h"
 #include "core/layout/LayoutBlockFlow.h"
 #include "core/layout/LayoutFullScreen.h"
 #include "core/layout/api/LayoutFullScreenItem.h"
@@ skipping 840 matching lines @@
       LayoutFullScreen::wrapLayoutObject(
           layoutObject, layoutObject ? layoutObject->parent() : 0, document());
     }
   }
 
   if (LocalFrame* frame = document()->frame()) {
     // TODO(foolip): Synchronize hover state changes with animation frames.
     // https://crbug.com/668758
     frame->eventHandler().scheduleHoverStateUpdate();
     frame->chromeClient().fullscreenElementChanged(fromElement, toElement);
+
+    if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled() &&
+        !RuntimeEnabledFeatures::rootLayerScrollingEnabled()) {
+      // Fullscreen status affects scroll paint properties through
+      // FrameView::userInputScrollable().
+      if (FrameView* frameView = frame->view())
+        frame->view()->setNeedsPaintPropertyUpdate();

[pdr., 2016/12/22 00:37:16]

nit: use the local variable:
if (FrameView* frameView = frame->view())
  frameView->setNeedsPaintPropertyUpdate();

[Xianzhu, 2016/12/22 00:45:29]

Done.

+    }
   }
 
   // TODO(foolip): This should not call updateStyleAndLayoutTree.
   document()->updateStyleAndLayoutTree();
 }
 
 DEFINE_TRACE(Fullscreen) {
   visitor->trace(m_pendingRequests);
   visitor->trace(m_fullscreenElementStack);
   Supplement<Document>::trace(visitor);
   ContextLifecycleObserver::trace(visitor);
 }
 
 }  // namespace blink