[Extensions] Convert some SyncExtensionFunctions

extensions/browser/api/test/test_api.h

 // 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.
 
 #ifndef EXTENSIONS_BROWSER_API_TEST_TEST_API_H_
 #define EXTENSIONS_BROWSER_API_TEST_TEST_API_H_
 
 #include "base/macros.h"
 #include "base/values.h"
 #include "extensions/browser/extension_function.h"
 
 namespace base {
 
 template <typename T>
 struct DefaultSingletonTraits;
 
 }  // namespace base
 
 namespace extensions {
 
 // A function that is only available in tests.
 // Prior to running, checks that we are in a testing process.
-class TestExtensionFunction : public SyncExtensionFunction {
+class TestExtensionFunction : public UIThreadExtensionFunction {
  protected:
   ~TestExtensionFunction() override;
 
-  // SyncExtensionFunction:
-  bool RunSync() override;
-
-  virtual bool RunSafe() = 0;
+  // ExtensionFunction:
+  bool PreRunValidation(std::string* error) override;
 };
 
 class TestNotifyPassFunction : public TestExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION("test.notifyPass", UNKNOWN)
 
  protected:
   ~TestNotifyPassFunction() override;
 
-  // TestExtensionFunction:
-  bool RunSafe() override;
+  // ExtensionFunction:
+  ResponseAction Run() override;
 };
 
 class TestNotifyFailFunction : public TestExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION("test.notifyFail", UNKNOWN)
 
  protected:
   ~TestNotifyFailFunction() override;
 
-  // TestExtensionFunction:
-  bool RunSafe() override;
+  // ExtensionFunction:
+  ResponseAction Run() override;
 };
 
 class TestLogFunction : public TestExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION("test.log", UNKNOWN)
 
  protected:
   ~TestLogFunction() override;
 
-  // TestExtensionFunction:
-  bool RunSafe() override;
+  // ExtensionFunction:
+  ResponseAction Run() override;
 };
 
 class TestSendMessageFunction : public UIThreadExtensionFunction {
  public:
   TestSendMessageFunction();
   DECLARE_EXTENSION_FUNCTION("test.sendMessage", UNKNOWN)
 
   // Sends a reply back to the calling extension. Many extensions don't need
   // a reply and will just ignore it.
   void Reply(const std::string& message);
@@ skipping 40 matching lines @@
     friend struct base::DefaultSingletonTraits<TestConfigState>;
     TestConfigState();
 
     base::DictionaryValue* config_state_;
 
     DISALLOW_COPY_AND_ASSIGN(TestConfigState);
   };
 
   ~TestGetConfigFunction() override;
 
-  // TestExtensionFunction:
-  bool RunSafe() override;
+  // ExtensionFunction:
+  ResponseAction Run() override;
 };
 
 class TestWaitForRoundTripFunction : public TestExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION("test.waitForRoundTrip", UNKNOWN)
 
  protected:
   ~TestWaitForRoundTripFunction() override;
 
-  // TestExtensionFunction:
-  bool RunSafe() override;
+  // ExtensionFunction:
+  ResponseAction Run() override;
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_API_TEST_TEST_API_H_