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

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: Fix merge conflicts. 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
« no previous file with comments | « chrome/browser/extensions/extension_prefs.cc ('k') | chrome/browser/extensions/extension_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extension_service.h
diff --git a/chrome/browser/extensions/extension_service.h b/chrome/browser/extensions/extension_service.h
index bf1c3d0e1bb328c93bbf66214f2e0234710b84a0..0ddd9c88accae20ec49693b176dedf054c9dd956 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
@@ -555,10 +545,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();
}
@@ -942,9 +928,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_;
« no previous file with comments | « chrome/browser/extensions/extension_prefs.cc ('k') | chrome/browser/extensions/extension_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698