[DevTools] Cleanup after reversing emulation flow.

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 "config.h"
 #include "web/DevToolsEmulator.h"
 
 #include "core/frame/FrameView.h"
 #include "core/frame/Settings.h"
 #include "core/page/Page.h"
 #include "platform/RuntimeEnabledFeatures.h"
-#include "public/web/WebDevToolsAgentClient.h"
 #include "public/web/WebDeviceEmulationParams.h"
 #include "web/WebDevToolsAgentImpl.h"
 #include "web/WebInputEventConversion.h"
 #include "web/WebLocalFrameImpl.h"
 #include "web/WebSettingsImpl.h"
 #include "web/WebViewImpl.h"
 
 namespace {
 
 static float calculateDeviceScaleAdjustment(int width, int height, float deviceScaleFactor)
@@ skipping 20 matching lines @@
     float ratio = static_cast<float>(minWidth - kWidthForMinFSM) / (kWidthForMaxFSM - kWidthForMinFSM);
     return ratio * (kMaxFSM - kMinFSM) + kMinFSM;
 }
 
 } // namespace
 
 namespace blink {
 
 DevToolsEmulator::DevToolsEmulator(WebViewImpl* webViewImpl)
     : m_webViewImpl(webViewImpl)
-    , m_devToolsAgent(nullptr)
     , m_deviceMetricsEnabled(false)
     , m_emulateMobileEnabled(false)
     , m_originalViewportEnabled(false)
     , m_isOverlayScrollbarsEnabled(false)
     , m_originalDefaultMinimumPageScaleFactor(0)
     , m_originalDefaultMaximumPageScaleFactor(0)
     , m_embedderTextAutosizingEnabled(webViewImpl->page()->settings().textAutosizingEnabled())
     , m_embedderDeviceScaleAdjustment(webViewImpl->page()->settings().deviceScaleAdjustment())
     , m_embedderPreferCompositingToLCDTextEnabled(webViewImpl->page()->settings().preferCompositingToLCDTextEnabled())
     , m_embedderUseMobileViewport(webViewImpl->page()->settings().useMobileViewportStyle())
     , m_touchEventEmulationEnabled(false)
     , m_originalTouchEnabled(false)
     , m_originalDeviceSupportsMouse(false)
     , m_originalDeviceSupportsTouch(false)
     , m_originalMaxTouchPoints(0)
-    , m_ignoreSetOverrides(false)
 {
 }
 
 DevToolsEmulator::~DevToolsEmulator()
 {
 }
 
-void DevToolsEmulator::setDevToolsAgent(WebDevToolsAgentImpl* agent)
-{
-    m_devToolsAgent = agent;
-}
-
 void DevToolsEmulator::setTextAutosizingEnabled(bool enabled)
 {
     m_embedderTextAutosizingEnabled = enabled;
     bool emulateMobileEnabled = m_deviceMetricsEnabled && m_emulateMobileEnabled;
     if (!emulateMobileEnabled)
         m_webViewImpl->page()->settings().setTextAutosizingEnabled(enabled);
 }
 
 void DevToolsEmulator::setDeviceScaleAdjustment(float deviceScaleAdjustment)
 {
@@ skipping 12 matching lines @@
 }
 
 void DevToolsEmulator::setUseMobileViewportStyle(bool enabled)
 {
     m_embedderUseMobileViewport = enabled;
     bool emulateMobileEnabled = m_deviceMetricsEnabled && m_emulateMobileEnabled;
     if (!emulateMobileEnabled)
         m_webViewImpl->page()->settings().setUseMobileViewportStyle(enabled);
 }
 
-void DevToolsEmulator::setDeviceMetricsOverride(int width, int height, float deviceScaleFactor, bool mobile, bool fitWindow, float scale, float offsetX, float offsetY)
-{
-    if (m_ignoreSetOverrides)
-        return;
-    WebDeviceEmulationParams params;
-    params.screenPosition = mobile ? WebDeviceEmulationParams::Mobile : WebDeviceEmulationParams::Desktop;
-    params.deviceScaleFactor = deviceScaleFactor;
-    params.viewSize = WebSize(width, height);
-    params.fitToView = fitWindow;
-    params.scale = scale;
-    params.offset = WebFloatPoint(offsetX, offsetY);
-
-    enableDeviceEmulationInner(params);
-    ASSERT(m_devToolsAgent);
-    m_devToolsAgent->client()->enableDeviceEmulation(params);
-
-    if (Document* document = m_webViewImpl->mainFrameImpl()->frame()->document()) {
-        document->styleResolverChanged();
-        document->mediaQueryAffectingValueChanged();
-    }
-}
-
 void DevToolsEmulator::enableDeviceEmulation(const WebDeviceEmulationParams& params)
 {
-    m_ignoreSetOverrides = true;
-    enableDeviceEmulationInner(params);
-    m_webViewImpl->setCompositorDeviceScaleFactorOverride(params.deviceScaleFactor);
-    m_webViewImpl->setRootLayerTransform(WebSize(params.offset.x, params.offset.y), params.scale);
-    if (Document* document = m_webViewImpl->mainFrameImpl()->frame()->document()) {
-        document->styleResolverChanged();
-        document->mediaQueryAffectingValueChanged();
-    }
-}
-
-void DevToolsEmulator::enableDeviceEmulationInner(const WebDeviceEmulationParams& params)
-{
     if (!m_deviceMetricsEnabled) {
         m_deviceMetricsEnabled = true;
         m_webViewImpl->setBackgroundColorOverride(Color::darkGray);
         m_webViewImpl->updateShowFPSCounterAndContinuousPainting();
-        if (m_devToolsAgent)
-            m_devToolsAgent->pageAgent()->setViewportNotificationsEnabled(true);
+        if (WebDevToolsAgentImpl* devToolsAgent = m_webViewImpl->devToolsAgentImpl())
+            devToolsAgent->pageAgent()->setViewportNotificationsEnabled(true);
     }
     m_webViewImpl->page()->settings().setDeviceScaleAdjustment(calculateDeviceScaleAdjustment(params.viewSize.width, params.viewSize.height, params.deviceScaleFactor));
 
     if (params.screenPosition == WebDeviceEmulationParams::Mobile)
         enableMobileEmulation();
     else
         disableMobileEmulation();
-}
 
-void DevToolsEmulator::clearDeviceMetricsOverride()
-{
-    if (m_ignoreSetOverrides)
-        return;
-    if (!m_deviceMetricsEnabled)
-        return;
-
-    disableDeviceEmulationInner();
-    ASSERT(m_devToolsAgent);
-    m_devToolsAgent->client()->disableDeviceEmulation();
+    m_webViewImpl->setCompositorDeviceScaleFactorOverride(params.deviceScaleFactor);
+    m_webViewImpl->setRootLayerTransform(WebSize(params.offset.x, params.offset.y), params.scale);
     if (Document* document = m_webViewImpl->mainFrameImpl()->frame()->document()) {
         document->styleResolverChanged();
         document->mediaQueryAffectingValueChanged();
     }
 }
 
 void DevToolsEmulator::disableDeviceEmulation()
 {
-    m_ignoreSetOverrides = true;
-    disableDeviceEmulationInner();
+    if (!m_deviceMetricsEnabled)
+        return;
+
+    m_deviceMetricsEnabled = false;
+    m_webViewImpl->setBackgroundColorOverride(Color::transparent);
+    m_webViewImpl->updateShowFPSCounterAndContinuousPainting();
+    if (WebDevToolsAgentImpl* devToolsAgent = m_webViewImpl->devToolsAgentImpl())
+        devToolsAgent->pageAgent()->setViewportNotificationsEnabled(false);
+    m_webViewImpl->page()->settings().setDeviceScaleAdjustment(m_embedderDeviceScaleAdjustment);
+    disableMobileEmulation();
     m_webViewImpl->setCompositorDeviceScaleFactorOverride(0.f);
     m_webViewImpl->setRootLayerTransform(WebSize(0.f, 0.f), 1.f);
     if (Document* document = m_webViewImpl->mainFrameImpl()->frame()->document()) {
         document->styleResolverChanged();
         document->mediaQueryAffectingValueChanged();
     }
 }
 
-void DevToolsEmulator::disableDeviceEmulationInner()
-{
-    if (m_deviceMetricsEnabled) {
-        m_deviceMetricsEnabled = false;
-        m_webViewImpl->setBackgroundColorOverride(Color::transparent);
-        m_webViewImpl->updateShowFPSCounterAndContinuousPainting();
-        if (m_devToolsAgent)
-            m_devToolsAgent->pageAgent()->setViewportNotificationsEnabled(false);
-        m_webViewImpl->page()->settings().setDeviceScaleAdjustment(m_embedderDeviceScaleAdjustment);
-        disableMobileEmulation();
-    }
-}
-
 void DevToolsEmulator::enableMobileEmulation()
 {
     if (m_emulateMobileEnabled)
         return;
     m_emulateMobileEnabled = true;
     m_isOverlayScrollbarsEnabled = RuntimeEnabledFeatures::overlayScrollbarsEnabled();
     RuntimeEnabledFeatures::setOverlayScrollbarsEnabled(true);
     m_originalViewportEnabled = RuntimeEnabledFeatures::cssViewportEnabled();
     RuntimeEnabledFeatures::setCSSViewportEnabled(true);
     m_webViewImpl->enableViewport();
@@ skipping 76 matching lines @@
             m_lastPinchAnchorCss.clear();
             m_lastPinchAnchorDip.clear();
         }
         return true;
     }
 
     return false;
 }
 
 } // namespace blink