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

Unified Diff: ui/message_center/message_center_bubble.cc

Issue 12052057: Introduces 'context' param to NotifierSettingsView. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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: ui/message_center/message_center_bubble.cc
diff --git a/ui/message_center/message_center_bubble.cc b/ui/message_center/message_center_bubble.cc
index 3ca2c0dd3c05759c22ac6861424351f684006141..f7f4af391a6be9e90037a706a966fa6f0ae011d6 100644
--- a/ui/message_center/message_center_bubble.cc
+++ b/ui/message_center/message_center_bubble.cc
@@ -223,6 +223,8 @@ class WebNotificationButtonView2 : public WebNotificationButtonViewBase,
// views::ButtonListener overrides:
virtual void ButtonPressed(views::Button* sender,
const ui::Event& event) OVERRIDE {
+ // TODO(mukai): Replace ShowNotificationSettings with
+ // ShowNotificationSettingsDialog(GetWidget()->GetNativeView())
if (sender == close_all_button())
list_delegate()->SendRemoveAllNotifications();
else if (sender == settings_button_)

Powered by Google App Engine
This is Rietveld 408576698