Refactor viewport initialization logic out of WebViewImpl.

Source/core/page/Settings.h

Index: Source/core/page/Settings.h
diff --git a/Source/core/page/Settings.h b/Source/core/page/Settings.h
index 3653fa03cfb238aaa7c62e7f353007441ded19e6..2207e78c26f2df728659fcc7a6860ca838999e69 100644
--- a/Source/core/page/Settings.h
+++ b/Source/core/page/Settings.h
@@ -97,6 +97,12 @@ public:
     void setTextAutosizingWindowSizeOverride(const IntSize&);
     const IntSize& textAutosizingWindowSizeOverride() const { return m_textAutosizingWindowSizeOverride; }
 
+    void setUseWideViewport(bool);
+    bool useWideViewport() const { return m_useWideViewport; }
+
+    void setLoadWithOverviewMode(bool);
+    bool loadWithOverviewMode() const { return m_loadWithOverviewMode; }
+
     // Only set by Layout Tests.
     void setMediaTypeOverride(const String&);
     const String& mediaTypeOverride() const { return m_mediaTypeOverride; }
@@ -172,6 +178,8 @@ private:
     float m_textAutosizingFontScaleFactor;
     IntSize m_textAutosizingWindowSizeOverride;
     bool m_textAutosizingEnabled : 1;
+    bool m_useWideViewport : 1;

[mnaganov (inactive), 2013/05/24 09:26:38]

You should also add them into Settings.in to get them properly initialized.

[mnaganov (inactive), 2013/05/24 09:29:26]

Ah, sorry. In your case, you should list them in the Settings constructor.

+    bool m_loadWithOverviewMode : 1;
 
     SETTINGS_MEMBER_VARIABLES