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

Unified Diff: ash/common/system/tray/tray_background_view.h

Issue 2618383003: Remove the extra separator appears on the right side of the shelf (Closed)
Patch Set: Created 3 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/common/system/tray/tray_background_view.h
diff --git a/ash/common/system/tray/tray_background_view.h b/ash/common/system/tray/tray_background_view.h
index 4788f2328529f1d1c890917424b10ca09e0bfafc..d3e66a641e3d34b604158aaa7855957b813406ec 100644
--- a/ash/common/system/tray/tray_background_view.h
+++ b/ash/common/system/tray/tray_background_view.h
@@ -128,6 +128,9 @@ class ASH_EXPORT TrayBackgroundView : public ActionableView,
// ShelfBackgroundAnimatorObserver:
void UpdateShelfItemBackground(int alpha) override;
+ // Updates the visibility of this tray's separator.
+ void SetSeparatorVisibility(bool is_show);
James Cook 2017/01/09 23:50:53 super nit: we usually use |visible| or just |show|
yiyix 2017/01/10 01:02:53 Done.
+
// Gets the anchor for bubbles, which is tray_container().
views::View* GetBubbleAnchor() const;
@@ -179,6 +182,10 @@ class ASH_EXPORT TrayBackgroundView : public ActionableView,
// in non-MD version and how the ink drop ripples behave in MD version.
bool is_active_;
+ // Visibility of this tray's separator which is a line of 1x32px and 4px to
+ // right of tray.
+ bool is_separator_visible_;
+
std::unique_ptr<TrayWidgetObserver> widget_observer_;
std::unique_ptr<TrayEventFilter> tray_event_filter_;

Powered by Google App Engine
This is Rietveld 408576698