Remove some UI->FILE->UI thread hops in ExecuteCodeFunction

extensions/browser/api/execute_code_function.h

Index: extensions/browser/api/execute_code_function.h
diff --git a/extensions/browser/api/execute_code_function.h b/extensions/browser/api/execute_code_function.h
index 3ba96e21317feb317c80eedc95e299434b9f52dd..ded8a40f505f272e5015da8e26b8f5901d523671 100644
--- a/extensions/browser/api/execute_code_function.h
+++ b/extensions/browser/api/execute_code_function.h
@@ -52,17 +52,23 @@ class ExecuteCodeFunction : public AsyncExtensionFunction {
   std::unique_ptr<api::extension_types::InjectDetails> details_;
 
  private:
-  // Called when contents from the file whose path is specified in JSON
-  // arguments has been loaded.
-  void DidLoadFile(bool success, std::unique_ptr<std::string> data);
-
-  // Runs on FILE thread. Loads message bundles for the extension and
-  // localizes the CSS data. Calls back DidLoadAndLocalizeFile on the UI thread.
-  void GetFileURLAndLocalizeCSS(ScriptExecutor::ScriptType script_type,
-                                std::unique_ptr<std::string> data,
-                                const std::string& extension_id,
-                                const base::FilePath& extension_path,
-                                const std::string& extension_default_locale);
+  // Retrieves the file url for the given |extension_path| and optionally
+  // localizes |data|.
+  // Localization depends on whether |might_require_localization| was specified.
+  // Only CSS file content needs to be localized.
+  void GetFileURLAndMaybeLocalizeOnFileThread(
+      const std::string& extension_id,
+      const base::FilePath& extension_path,
+      const std::string& extension_default_locale,
+      bool might_require_localization,
+      std::string* data);
+
+  void GetFileURLAndLocalizeComponentResourceOnFileThread(

[Devlin, 2016/09/01 06:15:21]

nit: a brief comment documenting this would be good.

[lazyboy, 2016/09/01 18:10:18]

Done.

+      std::unique_ptr<std::string> data,
+      bool might_require_localization,
+      const std::string& extension_id,
+      const base::FilePath& extension_path,
+      const std::string& extension_default_locale);
 
   // Run in UI thread.  Code string contains the code to be executed. Returns
   // true on success. If true is returned, this does an AddRef.