Add NotifyOnShowSettings implementation of notification provider API

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/message_center_settings_controller.h"
+#include "chrome/browser/profiles/profile_info_cache.h"
+#include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/common/chrome_version_info.h"
 #include "extensions/browser/event_router.h"
+#include "extensions/browser/extension_system.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 150 matching lines @@
 NotificationProviderNotifyOnShowSettingsFunction::
     ~NotificationProviderNotifyOnShowSettingsFunction() {
 }
 
 ExtensionFunction::ResponseAction
 NotificationProviderNotifyOnShowSettingsFunction::Run() {
   scoped_ptr<api::notification_provider::NotifyOnShowSettings::Params> params =
       api::notification_provider::NotifyOnShowSettings::Params::Create(*args_);
   EXTENSION_FUNCTION_VALIDATE(params.get());
 
+  ProfileInfoCache* profile_info_cache =
+      &g_browser_process->profile_manager()->GetProfileInfoCache();
+
+  scoped_ptr<MessageCenterSettingsController> settings_controller(
+      new MessageCenterSettingsController(profile_info_cache));

[dewittj, 2014/08/11 21:16:16]

I'd rather that you use the settings controller that already exists in the
NotificationUIManager instead of creating a new one here.

[liyanhou, 2014/08/12 17:15:00]

Done.

+
+  message_center::NotifierId notifier_id(
+      message_center::NotifierId::NotifierType::APPLICATION,
+      params->notifier_id);
+
+  bool has_advanced_settings =
+      settings_controller->NotifierHasAdvancedSettings(notifier_id);
+  if (has_advanced_settings)
+    settings_controller->OnNotifierAdvancedSettingsRequested(notifier_id, NULL);
   return RespondNow(ArgumentList(
-      api::notification_provider::NotifyOnShowSettings::Results::Create(true)));
+      api::notification_provider::NotifyOnShowSettings::Results::Create(
+          has_advanced_settings)));
 }
 
 NotificationProviderGetNotifierFunction::
     NotificationProviderGetNotifierFunction() {
 }
 
 NotificationProviderGetNotifierFunction::
     ~NotificationProviderGetNotifierFunction() {
 }
 
@@ skipping 15 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