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

Unified Diff: ash/system/tray/system_tray_bubble.cc

Issue 10546125: Add WebNotificationTray to the status area (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Make WebNotificationTray::Bubble a Widget::Observer; address nits Created 8 years, 6 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/tray/system_tray_bubble.cc
diff --git a/ash/system/tray/system_tray_bubble.cc b/ash/system/tray/system_tray_bubble.cc
index 51fd8f60569c08ff548fc45b27e4ed1e0026e40b..6329a5d571bd56cbc208fbefb6544802da8c87de 100644
--- a/ash/system/tray/system_tray_bubble.cc
+++ b/ash/system/tray/system_tray_bubble.cc
@@ -298,6 +298,7 @@ void SystemTrayBubble::InitView(const InitParams& init_params) {
void SystemTrayBubble::BubbleViewDestroyed() {
DestroyItemViews();
+ bubble_view_ = NULL;
stevenjb 2012/06/13 19:05:02 I added this here also, just to be safe.
}
gfx::Rect SystemTrayBubble::GetAnchorRect() const {
@@ -384,6 +385,19 @@ void SystemTrayBubble::Close() {
bubble_widget_->Close();
}
+void SystemTrayBubble::SetVisible(bool is_visible) {
+ if (!bubble_widget_)
+ return;
+ if (is_visible)
+ bubble_widget_->Show();
+ else
+ bubble_widget_->Hide();
+}
+
+bool SystemTrayBubble::IsVisible() {
+ return bubble_widget_ && bubble_widget_->IsVisible();
+}
+
void SystemTrayBubble::CreateItemViews(user::LoginStatus login_status) {
for (std::vector<ash::SystemTrayItem*>::iterator it = items_.begin();
it != items_.end();

Powered by Google App Engine
This is Rietveld 408576698