Fix webview.executeScript API

chrome/browser/extensions/api/webview/webview_api.cc

 // Copyright (c) 2013 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 "chrome/browser/extensions/api/webview/webview_api.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/api/browsing_data/browsing_data_api.h"
 #include "chrome/browser/extensions/api/context_menus/context_menus_api.h"
 #include "chrome/browser/extensions/api/context_menus/context_menus_api_helpers.h"
@@ skipping 245 matching lines @@
 bool WebviewExecuteCodeFunction::Init() {
   if (details_.get())
     return true;
 
   if (!args_->GetInteger(0, &guest_instance_id_))
     return false;
 
   if (!guest_instance_id_)
     return false;
 
+  if (!args_->GetString(1, &guest_src_))
+    return false;
+
+  if (guest_src_.empty()) {
+    return false;
+  }
+
   base::DictionaryValue* details_value = NULL;
-  if (!args_->GetDictionary(1, &details_value))
+  if (!args_->GetDictionary(2, &details_value))
     return false;
   scoped_ptr<InjectDetails> details(new InjectDetails());
   if (!InjectDetails::Populate(*details_value, details.get()))
     return false;
 
   details_ = details.Pass();
   return true;
 }
 
 bool WebviewExecuteCodeFunction::ShouldInsertCSS() const {
   return false;
 }
 
 bool WebviewExecuteCodeFunction::CanExecuteScriptOnPage() {
   return true;
 }
 
 extensions::ScriptExecutor* WebviewExecuteCodeFunction::GetScriptExecutor() {
   WebViewGuest* guest = WebViewGuest::From(
       render_view_host()->GetProcess()->GetID(), guest_instance_id_);
   if (!guest)
     return NULL;
 
   return guest->script_executor();
 }
 
 bool WebviewExecuteCodeFunction::IsWebView() const {
   return true;
 }
+std::string WebviewExecuteCodeFunction::GetWebViewSrc() const {

[Fady Samuel, 2014/04/04 19:35:05]

Move down one line.

[Xi Han, 2014/04/06 20:14:03]

Done.

+  return guest_src_;
+}
 
 WebviewExecuteScriptFunction::WebviewExecuteScriptFunction() {
 }
 
 void WebviewExecuteScriptFunction::OnExecuteCodeFinished(
     const std::string& error,
     int32 on_page_id,
     const GURL& on_url,
     const base::ListValue& result) {
   if (error.empty())
@@ skipping 221 matching lines @@
 
 WebviewTerminateFunction::~WebviewTerminateFunction() {
 }
 
 bool WebviewTerminateFunction::RunImplSafe(WebViewGuest* guest) {
   guest->Terminate();
   return true;
 }
 
 }  // namespace extensions