Expose a low-end device mode override flags for non-android OSs as well

base/base_switches.cc

Index: base/base_switches.cc
diff --git a/base/base_switches.cc b/base/base_switches.cc
index 81698cdfc13fdac9cc5f3ef49243e93ef6eab00a..5b93dbf174b313ed10fc4bde23013da884d3cb3e 100644
--- a/base/base_switches.cc
+++ b/base/base_switches.cc
@@ -9,11 +9,17 @@ namespace switches {
 // Disables the crash reporting.
 const char kDisableBreakpad[]               = "disable-breakpad";
 
+// Overrides low-end device detection, disabling low-end device optimizations.
+const char kDisableLowEndDeviceMode[]       = "disable-low-end-device-mode";
+
 // Indicates that crash reporting should be enabled. On platforms where helper
 // processes cannot access to files needed to make this decision, this flag is
 // generated internally.
 const char kEnableCrashReporter[]           = "enable-crash-reporter";
 
+// Overrides low-end device detection, enabling low-end device optimizations.
+const char kEnableLowEndDeviceMode[] = "enable-low-end-device-mode";

[brettw, 2014/06/11 21:02:40]

Can you align the = on this one and the next one you added for consistency?
(unless it makes it >80 cols)

[c.shu, 2014/06/12 21:12:48]

Done.

+
 // Generates full memory crash dump.
 const char kFullMemoryCrashReport[]         = "full-memory-crash-report";
 
@@ -60,12 +66,9 @@ const char kEnableCrashReporterForTesting[] =
     "enable-crash-reporter-for-testing";
 #endif
 
-#if defined(OS_ANDROID)
-// Overrides low-end device detection, disabling low-end device optimizations.
-const char kDisableLowEndDeviceMode[]       = "disable-low-end-device-mode";
-
-// Overrides low-end device detection, enabling low-end device optimizations.
-const char kEnableLowEndDeviceMode[]        = "enable-low-end-device-mode";
+#if !defined(OS_ANDROID)
+// Indicates whether to detect low-end device.
+const char kDetectLowEndDevice[] = "detect-low-end-device";

[brettw, 2014/06/11 21:02:40]

I'm not super excited about having 3 arguments to control this. Is it possible
to do something like:
--low-end-device-mode=0  meaning not low end
--low-end-device-mode=1  meaning low end
with undefined/empty meaning autodetect.

[c.shu, 2014/06/12 21:18:11]

I agree, brettw. I have a concern though the
enable-low-end-device/disable-low-end-device are already used on Android. Would
it cause some problem for some scripts if we changed the behavior?

 #endif
 
 }  // namespace switches