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

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

Issue 25366003: Moved some functions off ExtensionService into a new file extension_util. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Compile failures Created 7 years, 2 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 ee26e920283cd6e230691b9d64b3cb97eb30363d..4e93e3e7e12a36253a7b15586f4dc0d4daac4f8f 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -144,11 +144,6 @@ class ExtensionService
// Returns whether the URL is from either a hosted or packaged app.
bool IsInstalledApp(const GURL& url) const;
- // If the renderer is hosting an installed app with isolated storage,
- // returns it, otherwise returns NULL.
- const extensions::Extension* GetIsolatedAppForRenderer(
- int renderer_child_id) const;
-
// Attempts to uninstall an extension from a given ExtensionService. Returns
// true iff the target extension exists.
static bool UninstallExtensionHelper(ExtensionService* extensions_service,
@@ -186,11 +181,6 @@ class ExtensionService
extensions::ProcessMap* process_map() { return &process_map_; }
- // Whether this extension can run in an incognito window.
- virtual bool IsIncognitoEnabled(const std::string& extension_id) const;
- virtual void SetIsIncognitoEnabled(const std::string& extension_id,
- bool enabled);
-
// 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
@@ -199,19 +189,6 @@ class ExtensionService
const std::string& predecessor_extension_id,
const std::string& successor_extension_id);
- // Returns true if the given extension can see events and data from another
- // sub-profile (incognito to original profile, or vice versa).
- bool CanCrossIncognito(const extensions::Extension* extension) const;
-
- // Returns true if the given extension can be loaded in incognito.
- bool CanLoadInIncognito(const extensions::Extension* extension) const;
-
- // Whether this extension can inject scripts into pages with file URLs.
- bool AllowFileAccess(const extensions::Extension* extension) const;
- // Will reload the extension since this permission is applied at loading time
- // only.
- void SetAllowFileAccess(const extensions::Extension* extension, bool allow);
-
// Whether the persistent background page, if any, is ready. We don't load
// other components until then. If there is no background page, or if it is
// non-persistent (lazy), we consider it to be ready.
@@ -503,6 +480,7 @@ class ExtensionService
// TODO(skerner): Change to const ExtensionPrefs& extension_prefs() const,
// ExtensionPrefs* mutable_extension_prefs().
extensions::ExtensionPrefs* extension_prefs();
+ const extensions::ExtensionPrefs* extension_prefs() const;
extensions::SettingsFrontend* settings_frontend();
« no previous file with comments | « chrome/browser/extensions/extension_process_manager.cc ('k') | chrome/browser/extensions/extension_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698