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

Unified Diff: ash/system/user/tray_user.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/user/tray_user.cc
diff --git a/ash/system/user/tray_user.cc b/ash/system/user/tray_user.cc
index 5a07ea61f4e47b8354c2eb5ddb6030ebdbe8ecc0..897cebf43ebf2a4a4b1e198379ddda6047e0a4aa 100644
--- a/ash/system/user/tray_user.cc
+++ b/ash/system/user/tray_user.cc
@@ -1287,7 +1287,7 @@ void TrayUser::UpdateAfterLoginStatusChange(user::LoginStatus status) {
label_->SetText(bundle.GetLocalizedString(IDS_ASH_STATUS_TRAY_GUEST_LABEL));
}
- if (avatar_ && switches::UseAlternateShelfLayout()) {
+ if (avatar_) {
int corner_radius = GetTrayItemRadius();
avatar_->SetCornerRadii(0, corner_radius, corner_radius, 0);
avatar_->set_border(NULL);
@@ -1306,19 +1306,13 @@ void TrayUser::UpdateAfterShelfAlignmentChange(ShelfAlignment alignment) {
if (alignment == SHELF_ALIGNMENT_BOTTOM ||
alignment == SHELF_ALIGNMENT_TOP) {
if (avatar_) {
- if (switches::UseAlternateShelfLayout()) {
- if (multiprofile_index_) {
- avatar_->set_border(
- views::Border::CreateEmptyBorder(0, kTrayLabelSpacing, 0, 0));
- } else {
- avatar_->set_border(NULL);
- }
- avatar_->SetCornerRadii(0, corner_radius, corner_radius, 0);
+ if (multiprofile_index_) {
+ avatar_->set_border(
+ views::Border::CreateEmptyBorder(0, kTrayLabelSpacing, 0, 0));
} else {
- avatar_->set_border(views::Border::CreateEmptyBorder(
- 0, kTrayImageItemHorizontalPaddingBottomAlignment + 2,
- 0, kTrayImageItemHorizontalPaddingBottomAlignment));
+ avatar_->set_border(NULL);
}
+ avatar_->SetCornerRadii(0, corner_radius, corner_radius, 0);
}
if (label_) {
label_->set_border(views::Border::CreateEmptyBorder(
@@ -1330,17 +1324,13 @@ void TrayUser::UpdateAfterShelfAlignmentChange(ShelfAlignment alignment) {
0, 0, kUserLabelToIconPadding));
} else {
if (avatar_) {
- if (switches::UseAlternateShelfLayout()) {
- if (multiprofile_index_) {
- avatar_->set_border(
- views::Border::CreateEmptyBorder(kTrayLabelSpacing, 0, 0, 0));
- } else {
- avatar_->set_border(NULL);
- }
- avatar_->SetCornerRadii(0, 0, corner_radius, corner_radius);
+ if (multiprofile_index_) {
+ avatar_->set_border(
+ views::Border::CreateEmptyBorder(kTrayLabelSpacing, 0, 0, 0));
} else {
- SetTrayImageItemBorder(avatar_, alignment);
+ avatar_->set_border(NULL);
}
+ avatar_->SetCornerRadii(0, 0, corner_radius, corner_radius);
}
if (label_) {
label_->set_border(views::Border::CreateEmptyBorder(
@@ -1382,8 +1372,7 @@ void TrayUser::UpdateAvatarImage(user::LoginStatus status) {
GetTrayIndex() >= session_state_delegate->NumberOfLoggedInUsers())
return;
- int icon_size = switches::UseAlternateShelfLayout() ?
- kUserIconLargeSize : kUserIconSize;
+ int icon_size = kUserIconLargeSize;
avatar_->SetImage(
Shell::GetInstance()->session_state_delegate()->GetUserImage(

Powered by Google App Engine
This is Rietveld 408576698