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

Unified Diff: chrome/browser/notifications/application_notifier_source.cc

Issue 2064363002: Refactor MessageCenterSettingsController. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix misspell. Created 4 years, 6 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
Index: chrome/browser/notifications/application_notifier_source.cc
diff --git a/chrome/browser/notifications/application_notifier_source.cc b/chrome/browser/notifications/application_notifier_source.cc
new file mode 100644
index 0000000000000000000000000000000000000000..132b0f23bf0988e85b4cff652d14aa0174c1e032
--- /dev/null
+++ b/chrome/browser/notifications/application_notifier_source.cc
@@ -0,0 +1,81 @@
+// Copyright (c) 2016 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/application_notifier_source.h"
+
+#include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/notifications/notifier_state_tracker.h"
+#include "chrome/browser/notifications/notifier_state_tracker_factory.h"
+#include "chrome/browser/profiles/profile.h"
+#include "extensions/common/constants.h"
+#include "extensions/common/extension_set.h"
+#include "extensions/common/permissions/api_permission.h"
+#include "extensions/common/permissions/permissions_data.h"
+#include "ui/message_center/notifier_settings.h"
+
+ApplicationNotifierSource::ApplicationNotifierSource(Observer* observer)
+ : observer_(observer) {}
+
+ApplicationNotifierSource::~ApplicationNotifierSource() {}
+
+std::vector<std::unique_ptr<message_center::Notifier>>
+ApplicationNotifierSource::GetNotifierList(Profile* profile) {
+ std::vector<std::unique_ptr<message_center::Notifier>> notifiers;
+ const extensions::ExtensionSet& extension_set =
+ extensions::ExtensionRegistry::Get(profile)->enabled_extensions();
+ // The extension icon size has to be 32x32 at least to load bigger icons if
+ // the icon doesn't exist for the specified size, and in that case it falls
+ // back to the default icon. The fetched icon will be resized in the
+ // settings dialog. See chrome/browser/extensions/extension_icon_image.cc
+ // and crbug.com/222931
+ app_icon_loader_.reset(new extensions::ExtensionAppIconLoader(
+ profile, extension_misc::EXTENSION_ICON_SMALL, this));
+ for (extensions::ExtensionSet::const_iterator iter = extension_set.begin();
+ iter != extension_set.end(); ++iter) {
+ const extensions::Extension* extension = iter->get();
+ if (!extension->permissions_data()->HasAPIPermission(
+ extensions::APIPermission::kNotifications)) {
+ continue;
+ }
+
+ // Hosted apps are no longer able to affect the notifications permission
+ // state for web notifications.
+ // TODO(dewittj): Deprecate the 'notifications' permission for hosted
+ // apps.
+ if (extension->is_hosted_app())
+ continue;
+
+ message_center::NotifierId notifier_id(
+ message_center::NotifierId::APPLICATION, extension->id());
+ NotifierStateTracker* const notifier_state_tracker =
+ NotifierStateTrackerFactory::GetForProfile(profile);
+ notifiers.emplace_back(new message_center::Notifier(
+ notifier_id, base::UTF8ToUTF16(extension->name()),
+ notifier_state_tracker->IsNotifierEnabled(notifier_id)));
+ app_icon_loader_->FetchImage(extension->id());
+ }
+
+ return notifiers;
+}
+
+void ApplicationNotifierSource::SetNotifierEnabled(
+ Profile* profile,
+ const message_center::Notifier& notifier,
+ bool enabled) {
+ NotifierStateTrackerFactory::GetForProfile(profile)->SetNotifierEnabled(
+ notifier.notifier_id, enabled);
+ observer_->OnNotifierEnabledChanged(notifier.notifier_id, enabled);
+}
+
+message_center::NotifierId::NotifierType
+ApplicationNotifierSource::GetNotifierType() {
+ return message_center::NotifierId::APPLICATION;
+}
+
+void ApplicationNotifierSource::OnAppImageUpdated(const std::string& id,
+ const gfx::ImageSkia& image) {
+ observer_->OnIconImageUpdated(
+ message_center::NotifierId(message_center::NotifierId::APPLICATION, id),
+ gfx::Image(image));
+}

Powered by Google App Engine
This is Rietveld 408576698