[Extensions Bindings] Add support for updateArgumentsPreValidate

extensions/renderer/native_extension_bindings_system.cc

 // Copyright 2016 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/renderer/native_extension_bindings_system.h"
 
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
 #include "content/public/common/content_switches.h"
 #include "extensions/common/constants.h"
@@ skipping 64 matching lines @@
 void CallJsFunction(v8::Local<v8::Function> function,
                     v8::Local<v8::Context> context,
                     int argc,
                     v8::Local<v8::Value> argv[]) {
   ScriptContext* script_context =
       ScriptContextSet::GetContextByV8Context(context);
   CHECK(script_context);
   script_context->SafeCallFunction(function, argc, argv);
 }
 
+v8::Global<v8::Value> CallJsFunctionSync(v8::Local<v8::Function> function,
+                                         v8::Local<v8::Context> context,
+                                         int argc,
+                                         v8::Local<v8::Value> argv[]) {
+  bool did_complete = false;
+  v8::Global<v8::Value> result;
+  auto callback = base::Bind([](
+      v8::Isolate* isolate,
+      bool* did_complete_out,
+      v8::Global<v8::Value>* result_out,
+      const std::vector<v8::Local<v8::Value>>& results) {
+    *did_complete_out = true;
+    // The locals are released after the callback is executed, so we need to
+    // grab a persistent handle.
+    if (!results.empty() && !results[0].IsEmpty())
+      result_out->Reset(isolate, results[0]);
+  }, base::Unretained(context->GetIsolate()),
+     base::Unretained(&did_complete), base::Unretained(&result));
+
+  ScriptContext* script_context =
+      ScriptContextSet::GetContextByV8Context(context);
+  CHECK(script_context);
+  script_context->SafeCallFunction(function, argc, argv, callback);
+  CHECK(did_complete) << "expected script to execute synchronously";
+  return result;
+}
+
 // Returns the API schema indicated by |api_name|.
 const base::DictionaryValue& GetAPISchema(const std::string& api_name) {
   const base::DictionaryValue* schema =
       ExtensionAPI::GetSharedInstance()->GetSchema(api_name);
   CHECK(schema);
   return *schema;
 }
 
 // Returns true if the method specified by |method_name| is available to the
 // given |context|.
 bool IsAPIMethodAvailable(ScriptContext* context,
                           const std::string& method_name) {
   return context->GetAvailability(method_name).is_available();
 }
 
 }  // namespace
 
 NativeExtensionBindingsSystem::NativeExtensionBindingsSystem(
     const SendIPCMethod& send_ipc)
     : send_ipc_(send_ipc),
       api_system_(base::Bind(&CallJsFunction),
+                  base::Bind(&CallJsFunctionSync),
                   base::Bind(&GetAPISchema),
                   base::Bind(&NativeExtensionBindingsSystem::SendRequest,
                              base::Unretained(this))),
       weak_factory_(this) {}
 
 NativeExtensionBindingsSystem::~NativeExtensionBindingsSystem() {}
 
 void NativeExtensionBindingsSystem::DidCreateScriptContext(
     ScriptContext* context) {}
 
@@ skipping 174 matching lines @@
   params.has_callback = request->has_callback;
   params.user_gesture = request->has_user_gesture;
   // TODO(devlin): Make this work in ServiceWorkers.
   params.worker_thread_id = -1;
   params.service_worker_version_id = kInvalidServiceWorkerVersionId;
 
   send_ipc_.Run(script_context, params);
 }
 
 }  // namespace extensions