Remove apparently redundant uses of updatePageOverlays in WebViewImpl.

third_party/WebKit/Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 3808 matching lines @@
     }
 
     if (m_client) {
         if (m_layerTreeView) {
             updateLayerTreeViewport();
         } else {
             WebRect damagedRect(0, 0, m_size.width, m_size.height);
             m_client->widgetClient()->didInvalidateRect(damagedRect);
         }
     }
-    updatePageOverlays();
 }
 
 void WebViewImpl::configureAutoResizeMode()
 {
     if (!mainFrameImpl() || !mainFrameImpl()->frame() || !mainFrameImpl()->frame()->view())
         return;
 
     if (m_shouldAutoResize)
         mainFrameImpl()->frame()->view()->enableAutoSizeMode(m_minAutoSize, m_maxAutoSize);
     else
@@ skipping 241 matching lines @@
 
     FrameView* view = webframe->frame()->view();
 
     postLayoutResize(webframe);
 
     // Relayout immediately to avoid violating the rule that needsLayout()
     // isn't set at the end of a layout.
     if (view->needsLayout())
         view->layout();
 
+    updatePageOverlays();
+
     m_fullscreenController->didUpdateLayout();
-
     m_client->didUpdateLayout();
 }
 
 void WebViewImpl::didChangeContentsSize()
 {
     pageScaleConstraintsSet().didChangeContentsSize(contentsSize(), pageScaleFactor());
 }
 
 void WebViewImpl::pageScaleFactorChanged()
 {
@@ skipping 162 matching lines @@
     visualViewport.attachToLayerTree(layer);
     if (layer) {
         m_rootGraphicsLayer = visualViewport.rootGraphicsLayer();
         m_visualViewportContainerLayer = visualViewport.containerLayer();
         m_rootLayer = m_rootGraphicsLayer->platformLayer();
         updateRootLayerTransform();
         m_layerTreeView->setRootLayer(*m_rootLayer);
         // We register viewport layers here since there may not be a layer
         // tree view prior to this point.
         registerViewportLayersWithCompositor();
-        updatePageOverlays();
+
         // TODO(enne): Work around page visibility changes not being
         // propagated to the WebView in some circumstances.  This needs to
         // be refreshed here when setting a new root layer to avoid being
         // stuck in a presumed incorrectly invisible state.
         m_layerTreeView->setVisible(page()->isPageVisible());
     } else {
         m_rootGraphicsLayer = nullptr;
         m_visualViewportContainerLayer = nullptr;
         m_rootLayer = nullptr;
         // This means that we're transitioning to a new page. Suppress
         // commits until Blink generates invalidations so we don't
         // attempt to paint too early in the next page load.
         m_layerTreeView->setDeferCommits(true);
         m_layerTreeView->clearRootLayer();
         m_layerTreeView->clearViewportLayers();
     }
 }
 
 void WebViewImpl::invalidateRect(const IntRect& rect)
 {
-    if (m_layerTreeView)
+    if (m_layerTreeView) {
         updateLayerTreeViewport();
-    else if (m_client)
+    } else if (m_client) {
+        // This is only for WebViewPlugin.
         m_client->widgetClient()->didInvalidateRect(rect);
+    }
 }
 
 PaintLayerCompositor* WebViewImpl::compositor() const
 {
     WebLocalFrameImpl* frame = mainFrameImpl();
     if (!frame)
         return nullptr;
 
     Document* document = frame->frame()->document();
     if (!document || document->layoutViewItem().isNull())
@@ skipping 301 matching lines @@
         return nullptr;
     return focusedFrame;
 }
 
 LocalFrame* WebViewImpl::focusedLocalFrameAvailableForIme() const
 {
     return m_imeAcceptEvents ? focusedLocalFrameInWidget() : nullptr;
 }
 
 } // namespace blink