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

Unified Diff: ash/common/system/tray/system_tray.cc

Issue 2702583002: Remove the extra separator appears beside overview tray (Closed)
Patch Set: fix comments Created 3 years, 10 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
« no previous file with comments | « ash/common/system/overview/overview_button_tray.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/system/tray/system_tray.cc
diff --git a/ash/common/system/tray/system_tray.cc b/ash/common/system/tray/system_tray.cc
index 513635d4b547fb55deab63b87cdcef47bb8cd229..7986b14b1d08a3556e750579ae9a65a2410ae8b6 100644
--- a/ash/common/system/tray/system_tray.cc
+++ b/ash/common/system/tray/system_tray.cc
@@ -222,8 +222,9 @@ SystemTray::SystemTray(WmShelf* wm_shelf)
SetInkDropMode(InkDropMode::ON);
SetContentsBackground(false);
- // Since |system_tray| locates on the right most position, no separator is
- // required on its right side.
+ // Since user avatar is on the right hand side of System tray of a
+ // horizontal shelf and that is sufficient to indicate separation, no
+ // separator is required.
set_separator_visibility(false);
} else {
SetContentsBackground(true);
« no previous file with comments | « ash/common/system/overview/overview_button_tray.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698