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

Unified Diff: chrome/browser/extensions/extension_service.h

Issue 12680004: Remove chrome/ code to handle App Notifications (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Revert sync pref changes Created 7 years, 9 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/extensions/extension_service.h
diff --git a/chrome/browser/extensions/extension_service.h b/chrome/browser/extensions/extension_service.h
index c107c378152331a0102aeff26051e4fe86197691..d44ceef1618ec1988eb28d695370a92e71ae4178 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -54,7 +54,6 @@ class SequencedTaskRunner;
}
namespace extensions {
-class AppNotificationManager;
class AppSyncData;
class BrowserEventRouter;
class ComponentLoader;
@@ -229,15 +228,6 @@ class ExtensionService
virtual void SetIsIncognitoEnabled(const std::string& extension_id,
bool enabled);
- // When app notification setup is done, we call this to save the developer's
- // oauth client id which we'll need at uninstall time to revoke the oauth
- // permission grant for sending notifications.
- virtual void SetAppNotificationSetupDone(const std::string& extension_id,
- const std::string& oauth_client_id);
-
- virtual void SetAppNotificationDisabled(const std::string& extension_id,
- bool value);
-
// Updates the app launcher value for the moved extension so that it is now
// located after the given predecessor and before the successor. This will
// trigger a sync if needed. Empty strings are used to indicate no successor
@@ -557,10 +547,6 @@ class ExtensionService
extensions::MenuManager* menu_manager() { return &menu_manager_; }
- extensions::AppNotificationManager* app_notification_manager() {
- return app_notification_manager_.get();
- }
-
extensions::BrowserEventRouter* browser_event_router() {
return browser_event_router_.get();
}
@@ -944,9 +930,6 @@ class ExtensionService
// Keeps track of menu items added by extensions.
extensions::MenuManager menu_manager_;
- // Keeps track of app notifications.
- scoped_refptr<extensions::AppNotificationManager> app_notification_manager_;
-
// Flag to make sure event routers are only initialized once.
bool event_routers_initialized_;

Powered by Google App Engine
This is Rietveld 408576698