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

Unified Diff: chrome/common/extensions/extension.h

Issue 276074: Fix chrome_url_overides to work with packed extensions. (Closed)
Patch Set: review Created 11 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
« no previous file with comments | « chrome/browser/extensions/extension_dom_ui.cc ('k') | chrome/common/extensions/extension.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/extensions/extension.h
diff --git a/chrome/common/extensions/extension.h b/chrome/common/extensions/extension.h
index 0cb120ba964e4f3dd55f1280c043fd329992945c..8e4c94904c529d361f1f5d3abbf9e49eb8145500 100644
--- a/chrome/common/extensions/extension.h
+++ b/chrome/common/extensions/extension.h
@@ -26,6 +26,7 @@
class Extension {
public:
typedef std::vector<URLPattern> HostPermissions;
+ typedef std::map<const std::string, GURL> URLOverrideMap;
// What an extension was loaded from.
enum Location {
@@ -254,8 +255,8 @@ class Extension {
}
// Chrome URL overrides (see ExtensionOverrideUI).
- DictionaryValue* GetChromeURLOverrides() const {
- return chrome_url_overrides_.get();
+ const URLOverrideMap& GetChromeURLOverrides() const {
+ return chrome_url_overrides_;
}
// Runtime data:
@@ -376,7 +377,7 @@ class Extension {
// A map of chrome:// hostnames (newtab, downloads, etc.) to Extension URLs
// which override the handling of those URLs.
- scoped_ptr<DictionaryValue> chrome_url_overrides_;
+ URLOverrideMap chrome_url_overrides_;
// Runtime data:
« no previous file with comments | « chrome/browser/extensions/extension_dom_ui.cc ('k') | chrome/common/extensions/extension.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698