Make IsSolidColor() a property on CC scrollbar layers.

cc/base/switches.cc

Index: cc/base/switches.cc
diff --git a/cc/base/switches.cc b/cc/base/switches.cc
index 64d31da17f0748941ca361472474ba2e367a4595..6800fdbec6c2beadefdd9217507c8f65b2fc1a9e 100644
--- a/cc/base/switches.cc
+++ b/cc/base/switches.cc
@@ -131,6 +131,21 @@ const char kCCLayerTreeTestNoTimeout[] = "cc-layer-tree-test-no-timeout";
 const char kCCUnittestsTraceEventsToVLOG[] =
     "cc-unittests-trace-events-to-vlog";
 
+// Allow forcing of solid-color scrollbars

[jamesr, 2013/07/03 21:26:24]

what does it mean to "allow forcing"?  Does the flag force solid color
scrollbars on or not?

+const char kForceSolidColorScrollbars[] = "force-solid-color-scrollbars";
+
+// Solid-color scrollbar color in hex ARGB format, e.g. 0x80808080.
+// The 0x/0X prefix is optional.
+const char kSolidColorScrollbarColor[] = "solid-color-scrollbar-color";
+
+// Solid-color scrollbar thickness in DIP.
+const char kSolidColorScrollbarThickness[] = "solid-color-scrollbar-thickness";
+
+bool ForceSolidColorScrollbars() {

[jamesr, 2013/07/03 21:26:24]

the function name looks like it's doing something, not querying a value. can you
come up with a name that reads better? i expect typically calling code to look
like

if (Something()) { .... }

so you want the function name to make that line read naturally as

"if solid color scrollbars are forced on then ...."

as is, the line would be

if (ForceSolidColorScrollbars()) { .... }

which looks like the body of the if() is causing something to happen, not
querying a value

+  return CommandLine::ForCurrentProcess()->HasSwitch(
+      cc::switches::kForceSolidColorScrollbars);
+}
+
 bool IsImplSidePaintingEnabled() {
   const CommandLine& command_line = *CommandLine::ForCurrentProcess();