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

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

Issue 2723843002: Remove non-MD code from TrayBackgroundView (Closed)
Patch Set: Inverted flag + Added TODO 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
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 ba1381431294b4db073e463c5f3f3ecf8a1a42e2..7c0e67b4247d67833dc38099d562d6cf03fef61b 100644
--- a/ash/common/system/tray/system_tray.cc
+++ b/ash/common/system/tray/system_tray.cc
@@ -201,7 +201,7 @@ class SystemTray::ActivationObserver : public WmActivationObserver {
// SystemTray
SystemTray::SystemTray(WmShelf* wm_shelf)
- : TrayBackgroundView(wm_shelf),
+ : TrayBackgroundView(wm_shelf, true),
web_notification_tray_(nullptr),
detailed_item_(nullptr),
default_bubble_height_(0),
@@ -219,14 +219,11 @@ SystemTray::SystemTray(WmShelf* wm_shelf)
screen_share_tray_item_(nullptr) {
if (MaterialDesignController::IsShelfMaterial()) {
SetInkDropMode(InkDropMode::ON);
- SetContentsBackground(false);
// 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);
}
}

Powered by Google App Engine
This is Rietveld 408576698