Added flags to expose GCP->print preview integration

chrome/browser/about_flags.cc

Index: chrome/browser/about_flags.cc
diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index 3e7ec63a63624c3d37a072e22b39919344066db5..d26d57851312c89245f894ba28fb11899bbf5b9c 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -160,15 +160,32 @@ const Experiment kExperiments[] = {
     kOsAll,
     SINGLE_VALUE_TYPE(switches::kDisableGpuVsync)
   },
+#if !defined(GOOGLE_CHROME_BUILD) || defined(OS_CHROMEOS)

[Scott Byer, 2011/08/22 23:35:50]

There's a subtlety here with about flags and sync services - if the flag isn't
always present, it gets stripped from the flags to be set, leading to possible
strange behavior for users of multiple machines.

See the flag enablement for cloud-print-proxy - essentially zeroing out the OS
flag under the right cases keeps the flag  setting across platforms but doesn't
expose it.

[Albert Bodenhamer, 2011/08/23 01:05:07]

Done.

   // Only expose this for Chromium builds where users may not have the PDF
   // plugin. Do not give Google Chrome users the option to disable it here.
+  // Also expose it for Chrome OS where print preview is still experimental.
   {
     "print-preview",  // FLAGS:RECORD_UMA
+    IDS_FLAGS_CLOUD_PRINT_NAME,

[Scott Byer, 2011/08/22 23:35:50]

Are the IDS usages backwards between the two flags?

[Albert Bodenhamer, 2011/08/23 01:05:07]

Yikes!  Good catch.  A different logic error was masking the effects of this in
my testing.

+    IDS_FLAGS_CLOUD_PRINT_DESCRIPTION,
+    kOsAll,
+    SINGLE_VALUE_TYPE(switches::kEnablePrintPreview)
+  },
+#endif
+#if defined(GOOGLE_CHROME_BUILD) && !defined(OS_CHROMEOS)
+  // Define this for all CHROME builds (since print preview is on there)
+  // except for Chrome OS where cloud print is the only way to print and
+  // we shouldn't let users turn it off.
+  // If this flag is on and print preview is enabled, cloud print will be
+  // enabled in print preview
+  {
+    "cloud-print",  // FLAGS:RECORD_UMA
     IDS_FLAGS_PRINT_PREVIEW_NAME,
     IDS_FLAGS_PRINT_PREVIEW_DESCRIPTION,
     kOsAll,
-    SINGLE_VALUE_TYPE(switches::kEnablePrintPreview)
+    SINGLE_VALUE_TYPE(switches::kEnableCloudPrint)
   },
+#endif
   // TODO(dspringer): When NaCl is on by default, remove this flag entry.
   {
     "enable-nacl",  // FLAGS:RECORD_UMA