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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/extensions/crx_installer.cc

Issue 156843004: Remove ExtensionService::extension_prefs() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: additional cleanup Created 6 years, 10 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
« chrome/browser/extensions/api/omnibox/omnibox_api.cc ('K') | « chrome/browser/extensions/browser_action_apitest.cc ('k') | chrome/browser/extensions/crx_installer_browsertest.cc » ('j') | chrome/browser/extensions/extension_disabled_ui_browsertest.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/crx_installer.cc
diff --git a/chrome/browser/extensions/crx_installer.cc b/chrome/browser/extensions/crx_installer.cc
index dbaba4ed78fadecbe1eaecc886ba3567b62caffa..66a74fb4f36548e5c9a7dc2a5a5c8ece9eb04586 100644
--- a/chrome/browser/extensions/crx_installer.cc
+++ b/chrome/browser/extensions/crx_installer.cc
@@ -595,8 +595,8 @@ void CrxInstaller::ConfirmInstall() {
return;
}
- current_version_ =
- service->extension_prefs()->GetVersionString(extension()->id());
+ current_version_ = ExtensionPrefs::Get(service->profile())->
+ GetVersionString(extension()->id());
if (client_ &&
(!allow_silent_install_ || !approved_) &&
@@ -863,7 +863,7 @@ void CrxInstaller::ConfirmReEnable() {
if (!update_from_settings_page_)
return;
- ExtensionPrefs* prefs = service->extension_prefs();
+ ExtensionPrefs* prefs = ExtensionPrefs::Get(service->profile());
if (!prefs->DidExtensionEscalatePermissions(extension()->id()))
return;
« chrome/browser/extensions/api/omnibox/omnibox_api.cc ('K') | « chrome/browser/extensions/browser_action_apitest.cc ('k') | chrome/browser/extensions/crx_installer_browsertest.cc » ('j') | chrome/browser/extensions/extension_disabled_ui_browsertest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698