Apply WebSettings before initializing the main frame.

content/renderer/render_view_impl.cc

 // Copyright (c) 2012 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 "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/auto_reset.h"
@@ skipping 669 matching lines @@
     stats_collection_observer_.reset(new StatsCollectionObserver(this));
 
   // Debug cases of https://crbug.com/575245.
   base::debug::SetCrashKeyValue("rvinit_view_id",
                                 base::IntToString(routing_id()));
   base::debug::SetCrashKeyValue("rvinit_proxy_id",
                                 base::IntToString(params.proxy_routing_id));
   base::debug::SetCrashKeyValue(
       "rvinit_main_frame_id", base::IntToString(params.main_frame_routing_id));
 
+  webview()->setDisplayMode(display_mode_);
+  webview()->settings()->setPreferCompositingToLCDTextEnabled(
+      PreferCompositingToLCDText(compositor_deps_, device_scale_factor_));
+  webview()->settings()->setThreadedScrollingEnabled(
+      !command_line.HasSwitch(switches::kDisableThreadedScrolling));
+  webview()->settings()->setRootLayerScrolls(
+      command_line.HasSwitch(switches::kRootLayerScrolls));
+  webview()->setShowFPSCounter(
+      command_line.HasSwitch(cc::switches::kShowFPSCounter));
+
+  ApplyWebPreferencesInternal(webkit_preferences_, webview(), compositor_deps_);
+
+  if (switches::IsTouchDragDropEnabled())
+    webview()->settings()->setTouchDragDropEnabled(true);
+
+  WebSettings::SelectionStrategyType selection_strategy =
+      WebSettings::SelectionStrategyType::Character;
+  const std::string selection_strategy_str =
+      base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+          switches::kTouchTextSelectionStrategy);
+  if (selection_strategy_str == "direction")
+    selection_strategy = WebSettings::SelectionStrategyType::Direction;
+  webview()->settings()->setSelectionStrategy(selection_strategy);
+
+  ApplyBlinkSettings(command_line, webview()->settings());
+
   if (params.main_frame_routing_id != MSG_ROUTING_NONE) {
     main_render_frame_ = RenderFrameImpl::CreateMainFrame(
         this, params.main_frame_routing_id, params.main_frame_widget_routing_id,
         params.hidden, screen_info(), compositor_deps_, opener_frame);
   }
 
   if (params.proxy_routing_id != MSG_ROUTING_NONE) {
     CHECK(params.swapped_out);
     RenderFrameProxy::CreateFrameProxy(
         params.proxy_routing_id, routing_id(), params.opener_frame_route_id,
@@ skipping 22 matching lines @@
   if (RenderThreadImpl::current()) {
     RenderThreadImpl::current()->WidgetCreated();
     if (is_hidden_)
       RenderThreadImpl::current()->WidgetHidden();
   }
 
   // If this is a popup, we must wait for the CreatingNew_ACK message before
   // completing initialization.  Otherwise, we can finish it now.
   if (opener_id_ == MSG_ROUTING_NONE)
     did_show_ = true;
-  UpdateWebViewWithDeviceScaleFactor();
-  webview()->setDisplayMode(display_mode_);
-  webview()->settings()->setPreferCompositingToLCDTextEnabled(
-      PreferCompositingToLCDText(compositor_deps_, device_scale_factor_));
-  webview()->settings()->setThreadedScrollingEnabled(
-      !command_line.HasSwitch(switches::kDisableThreadedScrolling));
-  webview()->settings()->setRootLayerScrolls(
-      command_line.HasSwitch(switches::kRootLayerScrolls));
-  webview()->setShowFPSCounter(
-      command_line.HasSwitch(cc::switches::kShowFPSCounter));
-
-  ApplyWebPreferencesInternal(webkit_preferences_, webview(), compositor_deps_);
-
-  if (switches::IsTouchDragDropEnabled())
-    webview()->settings()->setTouchDragDropEnabled(true);
-
-  WebSettings::SelectionStrategyType selection_strategy =
-      WebSettings::SelectionStrategyType::Character;
-  const std::string selection_strategy_str =
-      base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
-          switches::kTouchTextSelectionStrategy);
-  if (selection_strategy_str == "direction")
-    selection_strategy = WebSettings::SelectionStrategyType::Direction;
-  webview()->settings()->setSelectionStrategy(selection_strategy);
 
   // Set the main frame's name.  Only needs to be done for WebLocalFrames,
   // since the remote case was handled as part of SetReplicatedState on the
   // proxy above.
   if (!params.replicated_frame_state.name.empty() &&
       webview()->mainFrame()->isWebLocalFrame()) {
     webview()->mainFrame()->setName(
         blink::WebString::fromUTF8(params.replicated_frame_state.name));
   }
 
   // TODO(davidben): Move this state from Blink into content.
   if (params.window_was_created_with_opener)
     webview()->setOpenedByDOM();
 
+  UpdateWebViewWithDeviceScaleFactor();

[dcheng, 2016/03/15 08:33:54]

One might wonder: why are these here?
Answer: this is the configuration that doesn't crash.

Unsatisfying, but this bit of code is essentially code soup. =(

   OnSetRendererPrefs(params.renderer_preferences);
 
-  ApplyBlinkSettings(command_line, webview()->settings());
-
   if (!params.enable_auto_resize) {
     OnResize(params.initial_size);
   } else {
     OnEnableAutoResize(params.min_size, params.max_size);
   }
 
 #if defined(OS_MACOSX)
   new TextInputClientObserver(this);
 #endif  // defined(OS_MACOSX)
 
@@ skipping 2742 matching lines @@
   if (IsUseZoomForDSFEnabled()) {
     webview()->setZoomFactorForDeviceScaleFactor(device_scale_factor_);
   } else {
     webview()->setDeviceScaleFactor(device_scale_factor_);
   }
   webview()->settings()->setPreferCompositingToLCDTextEnabled(
       PreferCompositingToLCDText(compositor_deps_, device_scale_factor_));
 }
 
 }  // namespace content