Invalidate touchEventTargetRects in ScrollingCoordinator when toggling touch emulation.

third_party/WebKit/Source/web/DevToolsEmulator.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "web/DevToolsEmulator.h"
 
 #include "core/frame/FrameHost.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/Settings.h"
 #include "core/page/Page.h"
+#include "core/page/scrolling/ScrollingCoordinator.h"
 #include "core/style/ComputedStyle.h"
 #include "platform/RuntimeEnabledFeatures.h"
 #include "public/platform/WebLayerTreeView.h"
 #include "web/WebInputEventConversion.h"
 #include "web/WebLocalFrameImpl.h"
 #include "web/WebSettingsImpl.h"
 #include "web/WebViewImpl.h"
 
 namespace {
 
@@ skipping 307 matching lines @@
         m_originalMaxTouchPoints = m_webViewImpl->page()->settings().maxTouchPoints();
     }
     RuntimeEnabledFeatures::setTouchEnabled(enabled ? true : m_originalTouchEnabled);
     if (!m_originalDeviceSupportsTouch) {
         m_webViewImpl->page()->settings().setDeviceSupportsMouse(enabled ? false : m_originalDeviceSupportsMouse);
         m_webViewImpl->page()->settings().setDeviceSupportsTouch(enabled ? true : m_originalDeviceSupportsTouch);
         // Currently emulation does not provide multiple touch points.
         m_webViewImpl->page()->settings().setMaxTouchPoints(enabled ? 1 : m_originalMaxTouchPoints);
     }
     m_touchEventEmulationEnabled = enabled;
+    if (ScrollingCoordinator* scrollingCoordinator = m_webViewImpl->page()->scrollingCoordinator()) {
+        // We only have to invalidate touch event target rects below, while using big hammer notifyGeometryChanged.
+        // But touch is not toggled often, and so it's fine to do more work vs introducing more fine-grained methods.
+        scrollingCoordinator->notifyGeometryChanged();
+        scrollingCoordinator->updateAfterCompositingChangeIfNeeded();
+    }
     // TODO(dgozman): mainFrameImpl() check in this class should be unnecessary.
     // It is only needed when we reattach and restore InspectorEmulationAgent,
     // which happens before everything has been setup correctly, and therefore
     // fails during remote -> local main frame transition.
     // We should instead route emulation from browser through the WebViewImpl
     // to the local main frame, and remove InspectorEmulationAgent entirely.
     if (m_webViewImpl->mainFrameImpl())
         m_webViewImpl->mainFrameImpl()->frameView()->layout();
 }
 
@@ skipping 32 matching lines @@
             m_lastPinchAnchorCss.reset();
             m_lastPinchAnchorDip.reset();
         }
         return true;
     }
 
     return false;
 }
 
 } // namespace blink