Adding implementation for MessageCenter::Delegate on MessageCeneterNotificationManager.

chrome/browser/notifications/message_center_notification_manager.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/notifications/message_center_notification_manager.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/extensions/extension_service.h"
+#include "chrome/browser/notifications/desktop_notification_service.h"
+#include "chrome/browser/notifications/desktop_notification_service_factory.h"
 #include "chrome/browser/notifications/notification.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/chrome_pages.h"
+#include "chrome/browser/ui/host_desktop.h"
 #include "chrome/common/extensions/extension_set.h"
 #include "chrome/common/pref_names.h"
 
 MessageCenterNotificationManager::MessageCenterNotificationManager(
     message_center::MessageCenter* message_center)
   : message_center_(message_center) {
   message_center_->SetDelegate(this);
 }
 
 MessageCenterNotificationManager::~MessageCenterNotificationManager() {
@@ skipping 105 matching lines @@
     }
   }
   return false;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // MessageCenter::Delegate
 
 void MessageCenterNotificationManager::DisableExtension(
     const std::string& notification_id) {
-  NOTIMPLEMENTED();
+  ProfileNotification* profile_notification =
+      FindProfileNotification(notification_id);
+  std::string extension_id = profile_notification->GetExtensionId();
+  DCHECK(!extension_id.empty()); // or UI should not have enabled the command.

[stevenjb, 2013/01/24 23:59:54]

two spaces before  //

[Dmitry Titov, 2013/01/25 00:47:45]

Done.

+  profile_notification->profile()->GetExtensionService()->DisableExtension(
+      extension_id, extensions::Extension::DISABLE_USER_ACTION);
 }
 
 void MessageCenterNotificationManager::DisableNotificationsFromSource(
     const std::string& notification_id) {
-  NOTIMPLEMENTED();
+  ProfileNotification* profile_notification =
+      FindProfileNotification(notification_id);
+  const GURL& source_url = profile_notification->notification().origin_url();
+  DCHECK(source_url.is_valid()); // or UI should not have enabled the command.

[stevenjb, 2013/01/24 23:59:54]

;  //

[Dmitry Titov, 2013/01/25 00:47:45]

Done.

+  DesktopNotificationService* service =
+      DesktopNotificationServiceFactory::GetForProfile(
+          profile_notification->profile());

[stevenjb, 2013/01/24 23:59:54]

nit: Don't need local variable 'source'

[Dmitry Titov, 2013/01/25 00:47:45]

Done.

+  service->DenyPermission(source_url);
 }
 
 void MessageCenterNotificationManager::NotificationRemoved(
     const std::string& notification_id) {
-  NOTIMPLEMENTED();
+  ProfileNotification* profile_notification =
+      FindProfileNotification(notification_id);

[stevenjb, 2013/01/24 23:59:54]

nit: Don't need local variable 

[Dmitry Titov, 2013/01/25 00:47:45]

Done.

+  RemoveProfileNotification(profile_notification);
 }
 
 void MessageCenterNotificationManager::ShowSettings(
     const std::string& notification_id) {
-  NOTIMPLEMENTED();
+  ProfileNotification* profile_notification =
+      FindProfileNotification(notification_id);
+  Profile* profile = profile_notification->profile();

[stevenjb, 2013/01/24 23:59:54]

nit: Don't need local variables

[Dmitry Titov, 2013/01/25 00:47:45]

Removed 'profile', left those used twice...

+  Browser* browser =
+      chrome::FindOrCreateTabbedBrowser(profile,
+                                        chrome::HOST_DESKTOP_TYPE_NATIVE);
+  if (profile_notification->GetExtensionId().empty())
+    chrome::ShowContentSettings(browser, CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  else
+    chrome::ShowExtensions(browser);
 }
 
 void MessageCenterNotificationManager::OnClicked(
     const std::string& notification_id) {
-  NOTIMPLEMENTED();
+  ProfileNotification* profile_notification =
+      FindProfileNotification(notification_id);

[stevenjb, 2013/01/24 23:59:54]

nit: Don't need local variable

[Dmitry Titov, 2013/01/25 00:47:45]

Done.

+  profile_notification->notification().Click();
 }
 
 void MessageCenterNotificationManager::OnButtonClicked(
     const std::string& notification_id, int button_index) {
-  NOTIMPLEMENTED();
+  ProfileNotification* profile_notification =
+      FindProfileNotification(notification_id);

[stevenjb, 2013/01/24 23:59:54]

nit: Don't need local variable

[Dmitry Titov, 2013/01/25 00:47:45]

Done.

+  profile_notification->notification().ButtonClick(button_index);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // ProfileNotification
 
 MessageCenterNotificationManager::ProfileNotification::ProfileNotification(
     Profile* profile, const Notification& notification)
   : profile_(profile),
     notification_(notification) {
   DCHECK(profile);
@@ skipping 30 matching lines @@
 }
 
 void MessageCenterNotificationManager::RemoveProfileNotification(
     ProfileNotification* profile_notification) {
   profile_notification->notification().Close(false); // Not by user.
   std::string id = profile_notification->notification().notification_id();
   message_center_->RemoveNotification(id);
   profile_notifications_.erase(id);
   delete profile_notification;
 }
+
+MessageCenterNotificationManager::ProfileNotification*
+    MessageCenterNotificationManager::FindProfileNotification(
+        const std::string& id) const {
+  NotificationMap::const_iterator iter = profile_notifications_.find(id);
+  if (iter == profile_notifications_.end()) {
+    NOTREACHED(); // If the notification is shown in UI, it must be in the map.

[stevenjb, 2013/01/24 23:59:54]

;  //

[Dmitry Titov, 2013/01/25 00:47:45]

Done.

+    return NULL;
+  }

[stevenjb, 2013/01/24 23:59:54]

This should be DCHECK(iter != profile_notifications_.end()) instead of using
NOTREACHED(). If we want to handle this case and return NULL here, use
LOG(ERROR) instead.

[Dmitry Titov, 2013/01/25 00:47:45]

Done.

+  DCHECK((*iter).second);
+  return (*iter).second;
+}