<webview>: Implement ExecuteScript

chrome/renderer/extensions/user_script_scheduler.cc

Index: chrome/renderer/extensions/user_script_scheduler.cc
diff --git a/chrome/renderer/extensions/user_script_scheduler.cc b/chrome/renderer/extensions/user_script_scheduler.cc
index 96da900f3382ddde587d7f081d6a5825f5b577bd..aa611fc75691c979dd8d60183a54ff96e2e5cd54 100644
--- a/chrome/renderer/extensions/user_script_scheduler.cc
+++ b/chrome/renderer/extensions/user_script_scheduler.cc
@@ -175,7 +175,8 @@ void UserScriptScheduler::ExecuteCodeImpl(
       //
       // For child frames, we just skip ones the extension doesn't have access
       // to and carry on.
-      if (!extension->CanExecuteScriptOnPage(child_frame->document().url(),
+      if (!extension->is_app() &&

[Matt Perry, 2013/01/23 02:50:49]

This is not quite the right check.

Are we sure we want to let an embedder call webview.executeScript() no matter
what URL is loaded into that webview? What if we required the webview app to
list its host permissions before it can execute script on those hosts?

If we really need to bypass this check, we should do it in a manner more
specific to webview. That might mean modifying ScriptExecutor to pass down an
extra is_webview param.

[Fady Samuel, 2013/01/23 20:12:03]

Done.

+          !extension->CanExecuteScriptOnPage(child_frame->document().url(),
                                              frame_->document().url(),
                                              extension_helper->tab_id(),
                                              NULL,