diff options
author | dgrogan@chromium.org <dgrogan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-01 01:17:21 +0000 |
---|---|---|
committer | dgrogan@chromium.org <dgrogan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-01 01:17:21 +0000 |
commit | 8834a3b3a4d52040e9e8436e20fa2dd0536b82a4 (patch) | |
tree | d4689f4bb451f3a306f059cf86978879c41cc38c /ash/ash_switches.cc | |
parent | 33b0299adb7277ab8f7df93a31a7c745fc915536 (diff) | |
download | chromium_src-8834a3b3a4d52040e9e8436e20fa2dd0536b82a4.zip chromium_src-8834a3b3a4d52040e9e8436e20fa2dd0536b82a4.tar.gz chromium_src-8834a3b3a4d52040e9e8436e20fa2dd0536b82a4.tar.bz2 |
Revert of Ash:Shelf - Cleanup of Alternate Shelf (part 1) (https://codereview.chromium.org/140323010/)
Reason for revert:
Broke Linux ChromiumOS Builder (dbg):
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Builder%20%28dbg%29/builds/45933/steps/compile/logs/stdio
FAILED: /b/build/goma/gomacc c++ -MMD -MF obj/ash/system/web_notification/ash.web_notification_tray.o.d ....
../../ash/system/web_notification/web_notification_tray.cc: In constructor 'ash::WebNotificationTray::WebNotificationTray(ash::internal::StatusAreaWidget*)':
../../ash/system/web_notification/web_notification_tray.cc:322:7:error: 'UseAlternateShelfLayout' is not a member of 'ash::switches'
../../ui/message_center/message_center_style.h: At global scope:
../../ui/message_center/message_center_style.h:103:15:error: 'message_center::kButtonSeparatorColor' defined but not used [-Werror=unused-variable]
../../ui/message_center/message_center_style.h:104:15:error: 'message_center::kHoveredButtonBackgroundColor' defined but not used [-Werror=unused-variable]
../../ui/message_center/message_center_style.h:111:15:error: 'message_center::kProgressBarBackgroundColor' defined but not used [-Werror=unused-variable]
../../ui/message_center/message_center_style.h:112:15:error: 'message_center::kProgressBarSliceColor' defined but not used [-Werror=unused-variable]
../../ui/message_center/message_center_style.h:43:15:error: 'message_center::settings::kEntrySeparatorColor' defined but not used [-Werror=unused-variable]
Original issue's description:
> Part 1) of cleaning up the code related to the shelf layout, specifically removing all instances of branching related to the 'alternate shelf layout' as this is now the only option.
> Removed the about:flags entry relating to alternate shelf layout.
>
> A patch will follow to simplify the remaining layout code as it is still a bit of a spaghetti like mess.
>
> R=jamescook@chromium.org, skuhne@chromium.org
> TBR=miket@chromium.org
> BUG=338429
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=254301
TBR=jamescook@chromium.org,skuhne@chromium.org,harrym@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=338429
Review URL: https://codereview.chromium.org/185143002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254306 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ash/ash_switches.cc')
-rw-r--r-- | ash/ash_switches.cc | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/ash/ash_switches.cc b/ash/ash_switches.cc index 2a010d6..bde14fe 100644 --- a/ash/ash_switches.cc +++ b/ash/ash_switches.cc @@ -148,6 +148,14 @@ 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: +// Slightly smaller profile, only 2 states for the "bar highlight" on +// launcher buttons, app list icon with more visible state indication, +// app list icon repositionable and defaulting as 1st item in shelf, +// more visible state indication for background on status area. +// crbug's [244983, 244990, 244994, 245005, 245012] +const char kAshUseAlternateShelfLayout[] = "ash-use-alternate-shelf"; + // Uses the 1st display in --ash-host-window-bounds as internal display. // This is for debugging on linux desktop. const char kAshUseFirstDisplayAsInternal[] = @@ -171,6 +179,11 @@ bool UseAlternateFrameCaptionButtonStyle() { HasSwitch(kAshEnableAlternateFrameCaptionButtonStyle); } +bool UseAlternateShelfLayout() { + return !CommandLine::ForCurrentProcess()-> + HasSwitch(kAshDisableAlternateShelfLayout); +} + bool UseDragOffShelf() { return !CommandLine::ForCurrentProcess()-> HasSwitch(kAshDisableDragOffShelf); |