Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(431)

Unified Diff: ash/system/web_notification/web_notification_tray.cc

Issue 140323010: Ash:Shelf - Cleanup of Alternate Shelf (part 1) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ButtonSize and ButtonSpacing moved to shelf_constants Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ash/system/web_notification/web_notification_tray.cc
diff --git a/ash/system/web_notification/web_notification_tray.cc b/ash/system/web_notification/web_notification_tray.cc
index 89c2deef1d8c4823b6da0ff5d6797c1188a59330..3ffd833fa38357e5cd6a0a0941b886e1e5956ba9 100644
--- a/ash/system/web_notification/web_notification_tray.cc
+++ b/ash/system/web_notification/web_notification_tray.cc
@@ -124,11 +124,10 @@ void WorkAreaObserver::SetSystemTrayHeight(int height) {
// should be reduced by the height of shelf's shown height.
if (shelf_ && shelf_->visibility_state() == SHELF_AUTO_HIDE &&
shelf_->auto_hide_state() == SHELF_AUTO_HIDE_SHOWN) {
- system_tray_height_ -= ShelfLayoutManager::GetPreferredShelfSize() -
- ShelfLayoutManager::kAutoHideSize;
+ system_tray_height_ -= kShelfSize - ShelfLayoutManager::kAutoHideSize;
}
- if (system_tray_height_ > 0 && ash::switches::UseAlternateShelfLayout())
+ if (system_tray_height_ > 0)
system_tray_height_ += message_center::kMarginBetweenItems;
if (!shelf_)
@@ -174,8 +173,7 @@ void WorkAreaObserver::OnAutoHideStateChanged(ShelfAutoHideState new_state) {
new_state == SHELF_AUTO_HIDE_SHOWN) {
// Since the work_area is already reduced by kAutoHideSize, the inset width
// should be just the difference.
- width = ShelfLayoutManager::GetPreferredShelfSize() -
- ShelfLayoutManager::kAutoHideSize;
+ width = kShelfSize - ShelfLayoutManager::kAutoHideSize;
}
work_area.Inset(shelf_->SelectValueForShelfAlignment(
gfx::Insets(0, 0, width, 0),
@@ -220,8 +218,7 @@ class WebNotificationBubbleWrapper {
}
views::TrayBubbleView* bubble_view = views::TrayBubbleView::Create(
tray->GetBubbleWindowContainer(), anchor, tray, &init_params);
- if (ash::switches::UseAlternateShelfLayout())
- bubble_view->SetArrowPaintType(views::BubbleBorder::PAINT_NONE);
+ bubble_view->SetArrowPaintType(views::BubbleBorder::PAINT_NONE);
bubble_wrapper_.reset(new TrayBubbleWrapper(tray, bubble_view));
bubble->InitializeContents(bubble_view);
}
@@ -338,7 +335,7 @@ bool WebNotificationTray::ShowMessageCenterInternal(bool show_settings) {
new message_center::MessageCenterBubble(
message_center(),
message_center_tray_.get(),
- ash::switches::UseAlternateShelfLayout());
+ true);
int max_height = 0;
aura::Window* status_area_window = status_area_widget()->GetNativeView();
@@ -409,7 +406,7 @@ bool WebNotificationTray::ShowPopups() {
internal::kShellWindowId_StatusContainer),
message_center(),
message_center_tray_.get(),
- ash::switches::UseAlternateShelfLayout()));
+ true));
work_area_observer_->StartObserving(
popup_collection_.get(), GetWidget()->GetNativeView()->GetRootWindow());
return true;

Powered by Google App Engine
This is Rietveld 408576698