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

Unified Diff: chrome/browser/automation/testing_automation_provider.cc

Issue 11365181: Remove GetExtensionService from Profile. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: replace missing extension_system include Created 8 years, 1 month 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/automation/testing_automation_provider.cc
diff --git a/chrome/browser/automation/testing_automation_provider.cc b/chrome/browser/automation/testing_automation_provider.cc
index 47036a383d9ad8c1c5622eaaae314a1f84401187..5e3f044a82e3f711ef4c57afe6f8032cd8297e28 100644
--- a/chrome/browser/automation/testing_automation_provider.cc
+++ b/chrome/browser/automation/testing_automation_provider.cc
@@ -3694,7 +3694,8 @@ void TestingAutomationProvider::InstallExtension(
}
args->GetBoolean("from_webstore", &from_webstore);
- ExtensionService* service = browser->profile()->GetExtensionService();
+ ExtensionService* service = extensions::ExtensionSystem::Get(
+ browser->profile())->extension_service();
ExtensionProcessManager* manager =
extensions::ExtensionSystem::Get(browser->profile())->process_manager();
if (service && manager) {
@@ -3775,7 +3776,8 @@ void TestingAutomationProvider::GetExtensionsInfo(DictionaryValue* args,
reply.SendError(error_msg);
return;
}
- ExtensionService* service = browser->profile()->GetExtensionService();
+ ExtensionService* service = extensions::ExtensionSystem::Get(
+ browser->profile())->extension_service();
if (!service) {
reply.SendError("No extensions service.");
return;
@@ -3850,7 +3852,8 @@ void TestingAutomationProvider::UninstallExtensionById(
AutomationJSONReply(this, reply_message).SendError(error);
return;
}
- ExtensionService* service = browser->profile()->GetExtensionService();
+ ExtensionService* service = extensions::ExtensionSystem::Get(
+ browser->profile())->extension_service();
if (!service) {
AutomationJSONReply(this, reply_message).SendError(
"No extensions service.");
@@ -3902,7 +3905,8 @@ void TestingAutomationProvider::SetExtensionStateById(
return;
}
- ExtensionService* service = browser->profile()->GetExtensionService();
+ ExtensionService* service = extensions::ExtensionSystem::Get(
+ browser->profile())->extension_service();
ExtensionProcessManager* manager =
extensions::ExtensionSystem::Get(browser->profile())->process_manager();
if (!service) {
@@ -4118,7 +4122,8 @@ void TestingAutomationProvider::UpdateExtensionsNow(
AutomationJSONReply(this, reply_message).SendError(error);
return;
}
- ExtensionService* service = browser->profile()->GetExtensionService();
+ ExtensionService* service = extensions::ExtensionSystem::Get(
+ browser->profile())->extension_service();
if (!service) {
AutomationJSONReply(this, reply_message).SendError(
"No extensions service.");
@@ -5015,7 +5020,8 @@ void TestingAutomationProvider::LaunchApp(
return;
}
- ExtensionService* service = browser->profile()->GetExtensionService();
+ ExtensionService* service = extensions::ExtensionSystem::Get(
+ browser->profile())->extension_service();
if (!service) {
AutomationJSONReply(this, reply_message).SendError(
"No extensions service.");
@@ -5073,7 +5079,8 @@ void TestingAutomationProvider::SetAppLaunchType(
return;
}
- ExtensionService* service = browser->profile()->GetExtensionService();
+ ExtensionService* service = extensions::ExtensionSystem::Get(
+ browser->profile())->extension_service();
if (!service) {
reply.SendError("No extensions service.");
return;
« no previous file with comments | « chrome/browser/automation/automation_util.cc ('k') | chrome/browser/background/background_application_list_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698