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

Unified Diff: ui/message_center/views/message_center_view_unittest.cc

Issue 114323002: Fixes the context menu for a notification. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix2 Created 7 years 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/views/message_center_view.cc ('k') | ui/message_center/views/message_popup_collection.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/message_center/views/message_center_view_unittest.cc
diff --git a/ui/message_center/views/message_center_view_unittest.cc b/ui/message_center/views/message_center_view_unittest.cc
index cd44e2dec66b73ebb9ff4c87928ab2d106d268cd..4e86ed3a087b61027ebed5af33028485b8345397 100644
--- a/ui/message_center/views/message_center_view_unittest.cc
+++ b/ui/message_center/views/message_center_view_unittest.cc
@@ -98,9 +98,9 @@ class MessageCenterViewTest : public testing::Test,
virtual void ClickOnNotification(const std::string& notification_id) OVERRIDE;
virtual void RemoveNotification(const std::string& notification_id,
bool by_user) OVERRIDE;
- virtual void DisableNotificationsFromThisSource(
- const NotifierId& notifier_id) OVERRIDE;
- virtual void ShowNotifierSettingsBubble() OVERRIDE;
+ virtual scoped_ptr<ui::MenuModel> CreateMenuModel(
+ const NotifierId& notifier_id,
+ const base::string16& display_source) OVERRIDE;
virtual bool HasClickedListener(const std::string& notification_id) OVERRIDE;
virtual void ClickOnNotificationButton(const std::string& notification_id,
int button_index) OVERRIDE;
@@ -191,15 +191,12 @@ void MessageCenterViewTest::RemoveNotification(
NOTREACHED();
}
-void MessageCenterViewTest::DisableNotificationsFromThisSource(
- const NotifierId& notifier_id) {
- // For this test, this method should not be invoked.
- NOTREACHED();
-}
-
-void MessageCenterViewTest::ShowNotifierSettingsBubble() {
+scoped_ptr<ui::MenuModel> MessageCenterViewTest::CreateMenuModel(
+ const NotifierId& notifier_id,
+ const base::string16& display_source) {
// For this test, this method should not be invoked.
NOTREACHED();
+ return scoped_ptr<ui::MenuModel>();
}
bool MessageCenterViewTest::HasClickedListener(
« no previous file with comments | « ui/message_center/views/message_center_view.cc ('k') | ui/message_center/views/message_popup_collection.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698