Rename (Chrome)SyncExtensionFunction::RunImpl to RunSync so that the RunImpl

extensions/browser/api/test/test_api.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "extensions/browser/api/test/test_api.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/memory/singleton.h"
@@ skipping 20 matching lines @@
 
 namespace extensions {
 
 namespace Log = core_api::test::Log;
 namespace NotifyFail = core_api::test::NotifyFail;
 namespace PassMessage = core_api::test::PassMessage;
 namespace WaitForRoundTrip = core_api::test::WaitForRoundTrip;
 
 TestExtensionFunction::~TestExtensionFunction() {}
 
-void TestExtensionFunction::Run() {
+bool TestExtensionFunction::RunSync() {
   if (!CommandLine::ForCurrentProcess()->HasSwitch(switches::kTestType)) {
     error_ = kNotTestProcessError;
-    SendResponse(false);
-    return;
+    return false;
   }
-  SendResponse(RunImpl());
+  return RunSafe();
 }
 
 TestNotifyPassFunction::~TestNotifyPassFunction() {}
 
-bool TestNotifyPassFunction::RunImpl() {
+bool TestNotifyPassFunction::RunSafe() {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_EXTENSION_TEST_PASSED,
       content::Source<content::BrowserContext>(dispatcher()->browser_context()),
       content::NotificationService::NoDetails());
   return true;
 }
 
 TestNotifyFailFunction::~TestNotifyFailFunction() {}
 
-bool TestNotifyFailFunction::RunImpl() {
+bool TestNotifyFailFunction::RunSafe() {
   scoped_ptr<NotifyFail::Params> params(NotifyFail::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_EXTENSION_TEST_FAILED,
       content::Source<content::BrowserContext>(dispatcher()->browser_context()),
       content::Details<std::string>(&params->message));
   return true;
 }
 
 TestLogFunction::~TestLogFunction() {}
 
-bool TestLogFunction::RunImpl() {
+bool TestLogFunction::RunSafe() {
   scoped_ptr<Log::Params> params(Log::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
   VLOG(1) << params->message;
   return true;
 }
 
 TestResetQuotaFunction::~TestResetQuotaFunction() {}
 
-bool TestResetQuotaFunction::RunImpl() {
+bool TestResetQuotaFunction::RunSafe() {
   QuotaService* quota =
       ExtensionSystem::Get(browser_context())->quota_service();
   quota->Purge();
   quota->violation_errors_.clear();
   return true;
 }
 
 bool TestSendMessageFunction::RunImpl() {
   scoped_ptr<PassMessage::Params> params(PassMessage::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
@@ skipping 22 matching lines @@
     : config_state_(NULL) {}
 
 // static
 TestGetConfigFunction::TestConfigState*
 TestGetConfigFunction::TestConfigState::GetInstance() {
   return Singleton<TestConfigState>::get();
 }
 
 TestGetConfigFunction::~TestGetConfigFunction() {}
 
-bool TestGetConfigFunction::RunImpl() {
+bool TestGetConfigFunction::RunSafe() {
   TestConfigState* test_config_state = TestConfigState::GetInstance();
 
   if (!test_config_state->config_state()) {
     error_ = kNoTestConfigDataError;
     return false;
   }
 
   SetResult(test_config_state->config_state()->DeepCopy());
   return true;
 }
 
 TestWaitForRoundTripFunction::~TestWaitForRoundTripFunction() {}
 
-bool TestWaitForRoundTripFunction::RunImpl() {
+bool TestWaitForRoundTripFunction::RunSafe() {
   scoped_ptr<WaitForRoundTrip::Params> params(
       WaitForRoundTrip::Params::Create(*args_));
   SetResult(new base::StringValue(params->message));
   return true;
 }
 
 }  // namespace extensions