Add NotifyOnPermissionLevelChanged implementation of notification

chrome/browser/extensions/api/notification_provider/notification_provider_api.cc

 // Copyright 2014 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/extensions/api/notification_provider/notification_provider_api.h"
 
 #include "base/callback.h"
 #include "base/guid.h"
 #include "base/rand_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.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/notifications/notification_ui_manager.h"
 #include "chrome/common/chrome_version_info.h"
 #include "extensions/browser/event_router.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/features/feature.h"
 #include "ui/base/layout.h"
+#include "ui/message_center/notifier_settings.h"
 #include "url/gurl.h"
 
 namespace extensions {
 
 NotificationProviderEventRouter::NotificationProviderEventRouter(
     Profile* profile)
     : profile_(profile) {
 }
 
 NotificationProviderEventRouter::~NotificationProviderEventRouter() {
@@ skipping 154 matching lines @@
 
 NotificationProviderNotifyOnPermissionLevelChangedFunction::
     ~NotificationProviderNotifyOnPermissionLevelChangedFunction() {
 }
 
 ExtensionFunction::ResponseAction
 NotificationProviderNotifyOnPermissionLevelChangedFunction::Run() {
   scoped_ptr<api::notification_provider::NotifyOnPermissionLevelChanged::Params>
       params = api::notification_provider::NotifyOnPermissionLevelChanged::
           Params::Create(*args_);
-
   EXTENSION_FUNCTION_VALIDATE(params.get());
 
+  bool is_application_type =

[Pete Williamson, 2014/08/13 19:54:01]

Let's add a comment about why we only check application type and not web type,
or at least move the comment below up to here.

[liyanhou, 2014/08/14 16:15:24]

Done.

+      (params->notifier_type ==
+       api::notification_provider::NotifierType::NOTIFIER_TYPE_APPLICATION);
+  if (is_application_type) {
+    bool enabled =
+        (params->level == api::notification_provider::NotifierPermissionLevel::
+                              NOTIFIER_PERMISSION_LEVEL_GRANTED)
+            ? true
+            : false;
+
+    DesktopNotificationService* desktop_notification_service =
+        DesktopNotificationServiceFactory::GetForProfile(GetProfile());
+    message_center::NotifierId notifier_id(
+        message_center::NotifierId::NotifierType::APPLICATION,
+        params->notifier_id);
+
+    desktop_notification_service->SetNotifierEnabled(notifier_id, enabled);
+  }
+
+  // Third party apps/extensions with notification provider API will not be able
+  // to change permission levels of web notifiers. Because the list of allowed
+  // websites should only be set in Chrome Settings manually by users. They can
+  // can change permission levels of application type notifiers.

[Pete Williamson, 2014/08/13 19:54:01]

nit: notifiers. Because -> notifiers, because

[liyanhou, 2014/08/14 16:15:24]

Done.

   return RespondNow(
       ArgumentList(api::notification_provider::NotifyOnPermissionLevelChanged::
-                       Results::Create(true)));
+                       Results::Create(is_application_type)));
 }
 
 NotificationProviderNotifyOnShowSettingsFunction::
     NotificationProviderNotifyOnShowSettingsFunction() {
 }
 
 NotificationProviderNotifyOnShowSettingsFunction::
     ~NotificationProviderNotifyOnShowSettingsFunction() {
 }
 
@@ skipping 33 matching lines @@
 
 ExtensionFunction::ResponseAction
 NotificationProviderGetAllNotifiersFunction::Run() {
   std::vector<linked_ptr<api::notification_provider::Notifier> > notifiers;
 
   return RespondNow(ArgumentList(
       api::notification_provider::GetAllNotifiers::Results::Create(notifiers)));
 }
 
 }  // namespace extensions