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

Side by Side Diff: ui/message_center/views/message_center_bubble.cc

Issue 12897004: Remove BubbleDelegateView::Show(); call GetWidget()->Show() instead; etc. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixup more BubbleDelegateView::Show() calls. Created 7 years, 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/message_center/views/message_center_bubble.h" 5 #include "ui/message_center/views/message_center_bubble.h"
6 6
7 #include <map> 7 #include <map>
8 8
9 #include "grit/ui_strings.h" 9 #include "grit/ui_strings.h"
10 #include "third_party/skia/include/core/SkPaint.h" 10 #include "third_party/skia/include/core/SkPaint.h"
(...skipping 469 matching lines...) Expand 10 before | Expand all | Expand 10 after
480 void MessageCenterBubble::OnBubbleViewDestroyed() { 480 void MessageCenterBubble::OnBubbleViewDestroyed() {
481 contents_view_ = NULL; 481 contents_view_ = NULL;
482 } 482 }
483 483
484 void MessageCenterBubble::UpdateBubbleView() { 484 void MessageCenterBubble::UpdateBubbleView() {
485 if (!bubble_view()) 485 if (!bubble_view())
486 return; // Could get called after view is closed 486 return; // Could get called after view is closed
487 const NotificationList::Notifications& notifications = 487 const NotificationList::Notifications& notifications =
488 message_center()->notification_list()->GetNotifications(); 488 message_center()->notification_list()->GetNotifications();
489 contents_view_->UpdateAllNotifications(notifications); 489 contents_view_->UpdateAllNotifications(notifications);
490 bubble_view()->Show(); 490 bubble_view()->GetWidget()->Show();
491 bubble_view()->UpdateBubble(); 491 bubble_view()->UpdateBubble();
492 } 492 }
493 493
494 void MessageCenterBubble::OnMouseEnteredView() { 494 void MessageCenterBubble::OnMouseEnteredView() {
495 } 495 }
496 496
497 void MessageCenterBubble::OnMouseExitedView() { 497 void MessageCenterBubble::OnMouseExitedView() {
498 } 498 }
499 499
500 size_t MessageCenterBubble::NumMessageViewsForTest() const { 500 size_t MessageCenterBubble::NumMessageViewsForTest() const {
501 return contents_view_->NumMessageViews(); 501 return contents_view_->NumMessageViews();
502 } 502 }
503 503
504 } // namespace message_center 504 } // namespace message_center
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698