Integrate new MD update menu icons.

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"
 
@@ skipping 930 matching lines @@
 const char kSSLKeyLogFile[]               = "ssl-key-log-file";
 
 // Does not show an infobar when an extension attaches to a page using
 // chrome.debugger page. Required to attach to extension background pages.
 const char kSilentDebuggerExtensionAPI[]    = "silent-debugger-extension-api";
 
 // Causes Chrome to launch without opening any windows by default. Useful if
 // one wishes to use Chrome as an ash server.
 const char kSilentLaunch[]                  = "silent-launch";
 
+// Simulates that bad modules have been found
+const char kSimulateBadModules[]            = "simulate-bad-modules";

[Peter Kasting, 2016/05/11 19:16:46]

We try to avoid switches if possible.  When they are necessary. they should be
for short-term debugging and there should be a concrete plan on when they'll be
removed.

Do you really need to add this, or have you done sufficient local testing to be
satisfied?  If there's a need to test the icon chosen when there are bad
modules, can that be tested by a unittest or similar instead of a switch?

[kylix_rd, 2016/05/11 19:54:43]

I wasn't clear on the philosophy of adding switches. This can certainly be
removed prior to the final commit version.

+
 // Simulates that elevation is needed to recover upgrade channel.
 const char kSimulateElevatedRecovery[]      = "simulate-elevated-recovery";
 
 // Simulates a critical update being available.
 const char kSimulateCriticalUpdate[]        = "simulate-critical-update";
 
 // Simulates that current version is outdated.
 const char kSimulateOutdated[]              = "simulate-outdated";
 
 // Simulates that current version is outdated and auto-update is off.
@@ skipping 394 matching lines @@
 
 // -----------------------------------------------------------------------------
 // DO NOT ADD YOUR CRAP 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