Enable <webview>.executeScript outside of Apps and Extensions [1]

extensions/browser/api/web_view/web_view_internal_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/web_view/web_view_internal_api.h"
 
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
@@ skipping 84 matching lines @@
     return false;
 
   base::DictionaryValue* details_value = NULL;
   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;
+
+  if (extension()) {
+    set_host_id(make_scoped_ptr(new HostID(HostID::EXTENSIONS,
+                                           extension()->id())));
+    return true;
+  }
+
+  auto rvh = render_view_host();

[Devlin, 2015/02/25 17:22:36]

Why use auto here?  It subtracts, rather than adds, to the readability.

[Xi Han, 2015/02/26 15:13:45]

Updated.

+  if (rvh) {
+    WebContents* web_contents = WebContents::FromRenderViewHost(rvh);
+    if (web_contents && web_contents->GetWebUI()) {
+      const GURL& url = rvh->GetSiteInstance()->GetSiteURL();
+      set_host_id(make_scoped_ptr(new HostID(HostID::WEBUI, url.spec())));
+      return true;
+    }
+  }
+  return false;
 }
 
 bool WebViewInternalExecuteCodeFunction::ShouldInsertCSS() const {
   return false;
 }
 
 bool WebViewInternalExecuteCodeFunction::CanExecuteScriptOnPage() {
   return true;
 }
 
@@ skipping 394 matching lines @@
   // Will finish asynchronously.
   return true;
 }
 
 void WebViewInternalClearDataFunction::ClearDataDone() {
   Release();  // Balanced in RunAsync().
   SendResponse(true);
 }
 
 }  // namespace extensions