Added flag for alternate launcher settings (remaining changes to come).

ash/ash_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 "ash/ash_switches.h"
 
 #include "base/command_line.h"
 
 namespace ash {
 namespace switches {
@@ skipping 102 matching lines @@
 
 // Specifies the layout mode and offsets for the secondary display for
 // testing. The format is "<t|r|b|l>,<offset>" where t=TOP, r=RIGHT,
 // b=BOTTOM and L=LEFT. For example, 'r,-100' means the secondary display
 // is positioned on the right with -100 offset. (above than primary)
 const char kAshSecondaryDisplayLayout[] = "ash-secondary-display-layout";
 
 // Enables the heads-up display for tracking touch points.
 const char kAshTouchHud[] = "ash-touch-hud";
 
+// Use alternate layout of the shelf for testing a new look and feel:
+// crbug's [244983, 244990, 244994, 245005, 245012]

[James Cook, 2013/06/13 15:55:30]

I would explain briefly the changes here, rather than referencing 5 different
bugs.  Just copy-paste the titles or something.  I usually only reference a
single bug, and only if it's a relatively complex issue.  (This avoids the
reader having to switch into their browser and look up the details of the
issue.)

[Harry McCleave, 2013/06/13 22:30:42]

Done.

+const char kAshUseAlternateShelfLayout[] = "ash-use-alt-shelf";

[James Cook, 2013/06/13 15:55:30]

Should this be ash-use-alternate-shelf to be more similar to the variable name? 
I hate abbreviations.

[Harry McCleave, 2013/06/13 22:30:42]

Done.

+
 // Uses the 1st display in --ash-host-window-bounds as internal display.
 // This is for debugging on linux desktop.
 const char kAshUseFirstDisplayAsInternal[] =
     "ash-use-first-display-as-internal";
 
 // (Most) Chrome OS hardware reports ACPI power button releases correctly.
 // Standard hardware reports releases immediately after presses.  If set, we
 // lock the screen or shutdown the system immediately in response to a press
 // instead of displaying an interactive animation.
 const char kAuraLegacyPowerButton[] = "aura-legacy-power-button";
@@ skipping 17 matching lines @@
   return !CommandLine::ForCurrentProcess()->
       HasSwitch(ash::switches::kAshDisableNewAudioHandler);
 }
 
 bool ShowAudioDeviceMenu() {
   return ash::switches::UseNewAudioHandler() &&
       CommandLine::ForCurrentProcess()->
           HasSwitch(ash::switches::kAshEnableAudioDeviceMenu);
 }
 
+bool UseAlternateShelfLayout() {

[James Cook, 2013/06/13 15:55:30]

I like this pattern of a UseFoo() function.

+  return CommandLine::ForCurrentProcess()->
+        HasSwitch(ash::switches::kAshUseAlternateShelfLayout);
+}
+
 }  // namespace switches
 }  // namespace ash