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

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: fix a compile error happened by a latest CL 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
« no previous file with comments | « ui/message_center/message_center.cc ('k') | ui/message_center/notification_list.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..b8a3207be77f1fb98f1c515c802c37c3293910f3 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 by
stevenjb 2013/01/25 00:18:05 s/by/with
Jun Mukai 2013/01/25 00:22:35 Done.
+ // ShowNotificationSettingsDialog(GetWidget()->GetNativeView())
if (sender == close_all_button())
list_delegate()->SendRemoveAllNotifications();
else if (sender == settings_button_)
« no previous file with comments | « ui/message_center/message_center.cc ('k') | ui/message_center/notification_list.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698