Add command line flags for compositor's default tile size and

content/browser/web_contents/web_contents_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/browser/web_contents/web_contents_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
@@ skipping 211 matching lines @@
       entry.transferred_global_request_id().child_id;
   params->transferred_request_request_id =
       entry.transferred_global_request_id().request_id;
   // Avoid downloading when in view-source mode.
   params->allow_download = !entry.IsViewSourceMode();
 
   if (delegate)
     delegate->AddNavigationHeaders(params->url, &params->extra_headers);
 }
 
+int GetSwitchValueInt(

[darin (slow to review), 2012/05/18 20:20:17]

nit: GetSwitchValueInt -> GetSwitchValueAsInt

+    const CommandLine& command_line,
+    const std::string& switch_string) {
+  std::string string_value = command_line.GetSwitchValueASCII(switch_string);
+  int int_value;

[darin (slow to review), 2012/05/18 20:20:17]

nit: initialize int_value?  or, check the return value of StringToInt.

+  base::StringToInt(string_value, &int_value);
+  return int_value;
+}
+
 }  // namespace
 
 namespace content {
 
 WebContents* WebContents::Create(
     BrowserContext* browser_context,
     SiteInstance* site_instance,
     int routing_id,
     const WebContents* base_web_contents,
     SessionStorageNamespace* session_storage_namespace) {
@@ skipping 286 matching lines @@
     gfx::Monitor monitor = gfx::Screen::GetMonitorNearestWindow(
         rvh->GetView()->GetNativeView());
     prefs.default_device_scale_factor =
         static_cast<int>(monitor.device_scale_factor());
   } else {
     prefs.default_device_scale_factor =
         gfx::Monitor::GetDefaultDeviceScaleFactor();;
   }
 #endif
 
+  if (command_line.HasSwitch(switches::kDefaultTileWidth))
+    prefs.default_tile_width = std::max(
+        1, GetSwitchValueInt(command_line, switches::kDefaultTileWidth));

[darin (slow to review), 2012/05/18 20:20:17]

nit: maybe the lower bound value should just be a parameter to
GetSwitchValueAsInt?

+  if (command_line.HasSwitch(switches::kDefaultTileHeight))
+    prefs.default_tile_height = std::max(
+        1, GetSwitchValueInt(command_line, switches::kDefaultTileHeight));
+  if (command_line.HasSwitch(switches::kMaxUntiledLayerWidth))
+    prefs.max_untiled_layer_width = std::max(
+        1, GetSwitchValueInt(command_line, switches::kMaxUntiledLayerWidth));
+  if (command_line.HasSwitch(switches::kMaxUntiledLayerHeight))
+    prefs.max_untiled_layer_height = std::max(
+        1, GetSwitchValueInt(command_line, switches::kMaxUntiledLayerHeight));
+
   content::GetContentClient()->browser()->OverrideWebkitPrefs(rvh, url, &prefs);
 
   return prefs;
 }
 
 NavigationControllerImpl& WebContentsImpl::GetControllerImpl() {
   return controller_;
 }
 
 RenderViewHostManager* WebContentsImpl::GetRenderManagerForTesting() {
@@ skipping 2235 matching lines @@
 void WebContentsImpl::CreateViewAndSetSizeForRVH(RenderViewHost* rvh) {
   RenderWidgetHostView* rwh_view = GetView()->CreateViewForWidget(rvh);
   // Can be NULL during tests.
   if (rwh_view)
     rwh_view->SetSize(GetView()->GetContainerSize());
 }
 
 RenderViewHostImpl* WebContentsImpl::GetRenderViewHostImpl() {
   return static_cast<RenderViewHostImpl*>(GetRenderViewHost());
 }