Add a new LaunchMode value for launches from a user experiment.

chrome/common/chrome_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 "chrome/common/chrome_switches.h"
 
 #include "base/base_switches.h"
 #include "base/command_line.h"
 #include "build/build_config.h"
 #include "ppapi/features/features.h"
@@ skipping 829 matching lines @@
 // to the PAC scripts.
 const char kUnsafePacUrl[] = "unsafe-pac-url";
 
 // A string used to override the default user agent with a custom one.
 const char kUserAgent[]                     = "user-agent";
 
 // Specifies the user data directory, which is where the browser will look for
 // all of its state.
 const char kUserDataDir[]                   = "user-data-dir";
 
+// Experimental. Shows a dialog asking the user to try chrome. This flag is to
+// be used only by the upgrade process.
+const char kUserExperimentUx[] = "user-experiment-ux";

[Peter Kasting, 2017/05/26 00:57:41]

I'm not terribly keen on a switch called "user experiment UX" because that
sounds like it applies to practically every UX change we make.

Can we talk about toasts, or launching, or something more descriptive in this
name?  Based on the comment, "show-try-chrome-dialog"?  Or, based on the enum
description in the next file, "show-installer-retention-ui"? (And a similar
rename of the variable.)

Nit: Indent the initialization part even with the lines above and below

[grt (UTC plus 2), 2017/05/29 07:08:42]

We may as well re-use the existing --try-chrome-again switch. Thanks.

+
 // Uses experimental simple cache backend if possible.
 const char kUseSimpleCacheBackend[]         = "use-simple-cache-backend";
 
 // Examines a .crx for validity and prints the result.
 const char kValidateCrx[]                   = "validate-crx";
 
 // Prints version information and quits.
 const char kVersion[]                       = "version";
 
 // Specify the initial window position: --window-position=x,y
@@ skipping 330 matching lines @@
 
 // -----------------------------------------------------------------------------
 // DO NOT ADD YOUR VERY NICE FLAGS TO THE BOTTOM OF THIS FILE.
 //
 // You were going to just dump your switches here, weren't you? Instead, please
 // put them in alphabetical order above, or in order inside the appropriate
 // ifdef at the bottom. The order should match the header.
 // -----------------------------------------------------------------------------
 
 }  // namespace switches