Shared Raster Worker Threads

cc/base/switches.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 "cc/base/switches.h"
 
 #include "base/command_line.h"
+#include "base/strings/string_number_conversions.h"
 
 namespace cc {
 namespace switches {
 
 // On platforms where checkerboards are used, prefer background colors instead
 // of checkerboards.
 const char kBackgroundColorInsteadOfCheckerboard[] =
     "background-color-instead-of-checkerboard";
 
 // Disables LCD text.
@@ skipping 161 matching lines @@
 #else
   return false;
 #endif
 }
 
 bool CheckGPURasterizationStatus() {
   const CommandLine& command_line = *CommandLine::ForCurrentProcess();
   return command_line.HasSwitch(switches::kEnableGPURasterization);
 }
 
+size_t CheckNumRasterThreads() {
+  const int kMinRasterThreads = 1;
+  const int kMaxRasterThreads = 64;
+  const int kDefaultNumRasterThreads = 1;
+
+  const CommandLine& command_line = *CommandLine::ForCurrentProcess();
+  int num_threads = kDefaultNumRasterThreads;

[reveman, 2013/12/28 14:45:48]

Please move this just above the "base::StringToInt(.." line now that you return
early.

[sohanjg, 2013/12/30 06:49:36]

Done.

+  if (command_line.HasSwitch(switches::kNumRasterThreads)) {
+    std::string string_value =
+        command_line.GetSwitchValueASCII(switches::kNumRasterThreads);
+    if ((base::StringToInt(string_value, &num_threads) &&
+        num_threads >= kMinRasterThreads && num_threads <= kMaxRasterThreads))
+        return num_threads;

[reveman, 2013/12/28 14:45:48]

nit: this line should be indented 2 spaces, not 4

[sohanjg, 2013/12/30 06:49:36]

Done.

+    LOG(WARNING) << "Failed to parse switch " <<

[reveman, 2013/12/28 14:45:48]

nit: maybe add a blank line between this and the above return statement

[sohanjg, 2013/12/30 06:49:36]

Done.

+                    switches::kNumRasterThreads  << ": " << string_value;
+  }
+  return num_threads;

[reveman, 2013/12/28 14:45:48]

nit: return kDefaultNumRasterThreads here instead

[sohanjg, 2013/12/30 06:49:36]

Done.

+}
+
 }  // namespace
 
 bool IsImplSidePaintingEnabled() {
   static bool enabled = CheckImplSidePaintingStatus();
   return enabled;
 }
 
 bool IsGPURasterizationEnabled() {
   static bool enabled = CheckGPURasterizationStatus();
   return enabled;
 }
 
 bool IsMapImageEnabled() {
   const CommandLine& command_line = *CommandLine::ForCurrentProcess();
 
   if (command_line.HasSwitch(switches::kDisableMapImage))
     return false;
   else if (command_line.HasSwitch(switches::kEnableMapImage))
     return true;
 
   return false;
 }
 
+size_t GetNumRasterThreads() {
+  static size_t num_raster_threads = CheckNumRasterThreads();

[reveman, 2013/12/28 14:45:48]

FYI, you'll have to update this if https://codereview.chromium.org/122063002
lands before this change.

[sohanjg, 2013/12/30 06:49:36]

OK.will take care of it, if that happens.

+  return num_raster_threads;
+}
+
 }  // namespace switches
 }  // namespace cc