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

Unified Diff: chrome/browser/extensions/api/management/management_api.cc

Issue 257333002: Drive extension functions from ExtensionFunction::Run. The (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase, fix test Created 6 years, 8 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/api/management/management_api.cc
diff --git a/chrome/browser/extensions/api/management/management_api.cc b/chrome/browser/extensions/api/management/management_api.cc
index 23844614398ad6aee7b7dac6cd2a57924320b000..5567f3ce91071f4aec5ed0079fe812f3d17d71a9 100644
--- a/chrome/browser/extensions/api/management/management_api.cc
+++ b/chrome/browser/extensions/api/management/management_api.cc
@@ -373,7 +373,7 @@ class SafeManifestJSONParser : public UtilityProcessHostClient {
} // namespace
-bool ManagementGetPermissionWarningsByManifestFunction::RunImpl() {
+bool ManagementGetPermissionWarningsByManifestFunction::RunAsync() {
scoped_ptr<management::GetPermissionWarningsByManifest::Params> params(
management::GetPermissionWarningsByManifest::Params::Create(*args_));
EXTENSION_FUNCTION_VALIDATE(params.get());
@@ -406,7 +406,7 @@ void ManagementGetPermissionWarningsByManifestFunction::OnParseSuccess(
management::GetPermissionWarningsByManifest::Results::Create(warnings);
SendResponse(true);
- // Matched with AddRef() in RunImpl().
+ // Matched with AddRef() in RunAsync().
Release();
}
@@ -415,7 +415,7 @@ void ManagementGetPermissionWarningsByManifestFunction::OnParseFailure(
error_ = error;
SendResponse(false);
- // Matched with AddRef() in RunImpl().
+ // Matched with AddRef() in RunAsync().
Release();
}
@@ -457,7 +457,7 @@ ManagementSetEnabledFunction::ManagementSetEnabledFunction() {
ManagementSetEnabledFunction::~ManagementSetEnabledFunction() {
}
-bool ManagementSetEnabledFunction::RunImpl() {
+bool ManagementSetEnabledFunction::RunAsync() {
scoped_ptr<management::SetEnabled::Params> params(
management::SetEnabled::Params::Create(*args_));
EXTENSION_FUNCTION_VALIDATE(params.get());
@@ -608,7 +608,7 @@ ManagementUninstallFunction::ManagementUninstallFunction() {
ManagementUninstallFunction::~ManagementUninstallFunction() {
}
-bool ManagementUninstallFunction::RunImpl() {
+bool ManagementUninstallFunction::RunAsync() {
scoped_ptr<management::Uninstall::Params> params(
management::Uninstall::Params::Create(*args_));
EXTENSION_FUNCTION_VALIDATE(extension_);
@@ -635,7 +635,7 @@ ManagementUninstallSelfFunction::ManagementUninstallSelfFunction() {
ManagementUninstallSelfFunction::~ManagementUninstallSelfFunction() {
}
-bool ManagementUninstallSelfFunction::RunImpl() {
+bool ManagementUninstallSelfFunction::RunAsync() {
scoped_ptr<management::UninstallSelf::Params> params(
management::UninstallSelf::Params::Create(*args_));
EXTENSION_FUNCTION_VALIDATE(params.get());

Powered by Google App Engine
This is Rietveld 408576698