Moved --top-chrome-md command line switch from chrome_switches to ui_base_switches.

ui/base/ui_base_switches.h

Index: ui/base/ui_base_switches.h
diff --git a/ui/base/ui_base_switches.h b/ui/base/ui_base_switches.h
index 5a7c20e7d87af546f3687ce7d17326b299f8782b..97c9c73be93493161e6c2933814eb99be23c117a 100644
--- a/ui/base/ui_base_switches.h
+++ b/ui/base/ui_base_switches.h
@@ -29,6 +29,14 @@ UI_BASE_EXPORT extern const char kEnableLinkDisambiguationPopup[];
 UI_BASE_EXPORT extern const char kEnableTouchDragDrop[];
 UI_BASE_EXPORT extern const char kEnableTouchEditing[];
 UI_BASE_EXPORT extern const char kLang[];
+
+#if defined(ENABLE_TOPCHROME_MD)
+UI_BASE_EXPORT extern const char kTopChromeMD[];
+UI_BASE_EXPORT extern const char kTopChromeMDMaterial[];
+UI_BASE_EXPORT extern const char kTopChromeMDMaterialHybrid[];
+UI_BASE_EXPORT extern const char kTopChromeMDNonMaterial[];

[Dan Beam, 2015/06/24 21:55:10]

I'm trying to combine MD-related flags.  Why do we need 4 for this?

[tdanderson, 2015/06/25 16:48:48]

As discussed offline, this is needed for our use case of switching between
non-material, material, and hybrid layouts at runtime.

[Dan Beam, 2015/06/25 17:05:35]

Yeah, I understand now why you need them.  Do you think we could add doc
comments here explaining the differences?  I think having 4 flags with the same
prefix is kind of uncommon.  This I arguably an easier spot than in some far
away .grd after finding which IDS_* that corresponds to which flag (if those
even exist for this).

[bruthig, 2015/06/25 17:49:49]

As is the convention for these switches, the documentation is in the .cc file.

As for having switches having the same prefix, this may not seem common when
looking at the different *_switches.h files because most aren't of the
MULTI_VALUE_TYPE.  I took a look at the different Experiment::Choice arrays in
about_flags.cc and some examples of this convention are: kTouchEventsChoices,
kMarkNonSecureAsChoices, kExtensionContentVerificationChoices,
kExtensionContentVerificationChoices.

I'm open to suggestions but I feel like it is currently following conventions.

[Dan Beam, 2015/06/25 18:11:43]

you're right and I just probably haven't seen that many MULTI_VALUE_TYPE.  good
enough for me, sorry for the noise.

+#endif  // defind(ENABLE_TOPCHROME_MD)
+
 UI_BASE_EXPORT extern const char kViewerConnect[];
 
 }  // namespace switches