Settings should not call into inspector

Source/core/inspector/InspectorPageAgent.cpp

 /*
  * Copyright (C) 2011 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 764 matching lines @@
 }
 
 void InspectorPageAgent::getScriptExecutionStatus(ErrorString*, PageCommandHandler::Result::Enum* status)
 {
     bool disabledByScriptController = false;
     bool disabledInSettings = false;
     Frame* frame = mainFrame();
     if (frame) {
         disabledByScriptController = !frame->script().canExecuteScripts(NotAboutToExecuteScript);
         if (frame->settings())
-            disabledInSettings = !frame->settings()->isScriptEnabled();
+            disabledInSettings = !frame->settings()->scriptEnabled();
     }
 
     if (!disabledByScriptController) {
         *status = PageCommandHandler::Result::Allowed;
         return;
     }
 
     if (disabledInSettings)
         *status = PageCommandHandler::Result::Disabled;
     else
@@ skipping 256 matching lines @@
 }
 
 void InspectorPageAgent::scriptsEnabled(bool isEnabled)
 {
     if (m_ignoreScriptsEnabledNotification)
         return;
 
     m_frontend->scriptsEnabled(isEnabled);
 }
 
+void InspectorPageAgent::textAutosizingEnabled(bool isEnabled)
+{
+    if (!m_deviceMetricsOverridden)
+        m_settingsCacheTextAutosizingEnabled = isEnabled;
+}
+
+void InspectorPageAgent::fontScaleFactor(float factor)
+{
+    if (!m_deviceMetricsOverridden)
+        m_settingsCacheFontScaleFactor = factor;
+}
+
 PassRefPtr<TypeBuilder::Page::Frame> InspectorPageAgent::buildObjectForFrame(Frame* frame)
 {
     RefPtr<TypeBuilder::Page::Frame> frameObject = TypeBuilder::Page::Frame::create()
         .setId(frameId(frame))
         .setLoaderId(loaderId(frame->loader().documentLoader()))
         .setUrl(urlWithoutFragment(frame->document()->url()).string())
         .setMimeType(frame->loader().documentLoader()->responseMIMEType())
         .setSecurityOrigin(frame->document()->securityOrigin()->toRawString());
     if (frame->tree().parent())
         frameObject->setParentId(frameId(frame->tree().parent()));
@@ skipping 50 matching lines @@
     }
     return result;
 }
 
 void InspectorPageAgent::updateViewMetrics(int width, int height, double deviceScaleFactor, bool emulateViewport, bool fitWindow)
 {
     if (width && height && !m_page->settings().acceleratedCompositingEnabled())
         return;
 
     m_deviceMetricsOverridden = width && height;
+    Settings& settings = m_page->settings();
+    if (m_deviceMetricsOverridden) {
+        settings.setTextAutosizingEnabled(overrideTextAutosizing());
+        settings.setDeviceScaleAdjustment(overrideFontScaleFactor());
+    } else {
+        settings.setTextAutosizingEnabled(m_settingsCacheTextAutosizingEnabled);
+        settings.setDeviceScaleAdjustment(m_settingsCacheFontScaleFactor);
+    }
+
     m_emulateViewportEnabled = emulateViewport;
     m_client->overrideDeviceMetrics(width, height, static_cast<float>(deviceScaleFactor), emulateViewport, fitWindow);
 
     Document* document = mainFrame()->document();
     if (document) {
         document->styleResolverChanged(RecalcStyleImmediately);
         document->mediaQueryAffectingValueChanged();
     }
     InspectorInstrumentation::mediaQueryResultChanged(document);
 
@@ skipping 61 matching lines @@
     }
 
     controller->didChangeDeviceOrientation(DeviceOrientationData::create(true, alpha, true, beta, true, gamma).get());
 }
 
 void InspectorPageAgent::clearDeviceOrientationOverride(ErrorString* error)
 {
     setDeviceOrientationOverride(error, 0, 0, 0);
 }
 
-bool InspectorPageAgent::overrideTextAutosizing(bool textAutosizing)
+bool InspectorPageAgent::overrideTextAutosizing()
 {
-    if (!m_deviceMetricsOverridden)
-        return textAutosizing;
     return m_state->getBoolean(PageAgentState::pageAgentTextAutosizingOverride);
 }
 
-float InspectorPageAgent::overrideFontScaleFactor(float fontScaleFactor)
+float InspectorPageAgent::overrideFontScaleFactor()
 {
-    if (!m_deviceMetricsOverridden)
-        return fontScaleFactor;
     return static_cast<float>(m_state->getDouble(PageAgentState::fontScaleFactor));
 }
 
 void InspectorPageAgent::setTouchEmulationEnabled(ErrorString*, bool enabled)
 {
     if (m_state->getBoolean(PageAgentState::touchEventEmulationEnabled) == enabled)
         return;
     updateTouchEventEmulationInPage(enabled);
 }
 
@@ skipping 82 matching lines @@
 }
 
 void InspectorPageAgent::setShowViewportSizeOnResize(ErrorString*, bool show, const bool* showGrid)
 {
     m_state->setBoolean(PageAgentState::showSizeOnResize, show);
     m_state->setBoolean(PageAgentState::showGridOnResize, showGrid && *showGrid);
 }
 
 } // namespace WebCore