Generic version of caching cc switches to avoid searching of switches on each function call

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"
 
 namespace cc {
 namespace switches {
@@ skipping 147 matching lines @@
   else if (command_line->HasSwitch(cc::switches::kEnableLCDText))
     return true;
 
 #if defined(OS_ANDROID)
   return false;
 #else
   return true;
 #endif
 }
 
-namespace {
-bool CheckImplSidePaintingStatus() {
+// impl side painting

[danakj, 2013/11/11 17:57:53]

The variable is called impl_side_painting, so I don't think this comment adds
much value here.

+static bool implSidePaintingStatus;

[danakj, 2013/11/11 17:57:53]

this is blink style, you want g_impl_side_painting_status

how about g_impl_side_painting_enabled though? 

+// map-image flag

[danakj, 2013/11/11 17:57:53]

same with this comment

+static bool mapImageStatus;
+
+static void InitializeCCSwitchesIfRequired() {
+  static bool has_initialized_ccswitches = false;
+  if (has_initialized_ccswitches)
+    return;
+
   const CommandLine& command_line = *CommandLine::ForCurrentProcess();
+  // impl-side painting

[danakj, 2013/11/11 17:57:53]

// Impl-side painting.

(capitals and punctuation in comments please)

+  if (command_line.HasSwitch(cc::switches::kDisableImplSidePainting)) {
+    implSidePaintingStatus = false;
+  } else if (command_line.HasSwitch(cc::switches::kEnableImplSidePainting)) {
+    implSidePaintingStatus = true;
+  } else {
+#if defined(OS_ANDROID)
+    implSidePaintingStatus = true;
+#else
+    implSidePaintingStatus = false;
+#endif
+  }
 
-  if (command_line.HasSwitch(cc::switches::kDisableImplSidePainting))
-    return false;
-  else if (command_line.HasSwitch(cc::switches::kEnableImplSidePainting))
-    return true;
+  // map-image flag

[danakj, 2013/11/11 17:57:53]

// Map-image flag.

+  if (command_line.HasSwitch(cc::switches::kDisableMapImage))
+    mapImageStatus = false;
+  else if (command_line.HasSwitch(cc::switches::kEnableMapImage))
+    mapImageStatus = true;
 
-#if defined(OS_ANDROID)
-  return true;
-#else
-  return false;
-#endif
+  has_initialized_ccswitches = true;
 }
-}  // namespace
 
 bool IsImplSidePaintingEnabled() {
-  static bool enabled = CheckImplSidePaintingStatus();
-  return enabled;
+  InitializeCCSwitchesIfRequired();
+  return implSidePaintingStatus;
 }
 
 bool IsMapImageEnabled() {
-  const CommandLine& command_line = *CommandLine::ForCurrentProcess();
-
-  if (command_line.HasSwitch(cc::switches::kDisableMapImage))
-    return false;
-  else if (command_line.HasSwitch(cc::switches::kEnableMapImage))
-    return true;
-
-  return false;
+  InitializeCCSwitchesIfRequired();
+  return mapImageStatus;
 }
 
 }  // namespace switches
 }  // namespace cc