Make SearchBox extension call through WebFrame to execute page script

chrome/renderer/searchbox_extension.cc

 // Copyright (c) 2011 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/renderer/searchbox_extension.h"
 
 #include <string>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 41 matching lines @@
     "    this.setSuggestions = function(text) {"
     "      SetSuggestions(text);"
     "    };"
     "    this.onchange = null;"
     "    this.onsubmit = null;"
     "    this.oncancel = null;"
     "    this.onresize = null;"
     "  };"
     "}";
 
-static const char kChangeEventName[] = "chrome.searchBox.onchange";
+static const char kDispatchChangeEventScript[] =
+    "if (window.chrome &&"
+    "    window.chrome.searchBox &&"
+    "    window.chrome.searchBox.onchange &&"
+    "    typeof window.chrome.searchBox.onchange == 'function') {"
+    "  window.chrome.searchBox.onchange();"
+    "  true;"
+    "}";
 
-static const char kSubmitEventName[] = "chrome.searchBox.onsubmit";
+static const char kDispatchSubmitEventScript[] =
+    "if (window.chrome &&"
+    "    window.chrome.searchBox &&"
+    "    window.chrome.searchBox.onsubmit &&"
+    "    typeof window.chrome.searchBox.onsubmit == 'function') {"
+    "  window.chrome.searchBox.onsubmit();"
+    "  true;"
+    "}";
 
-static const char kCancelEventName[] = "chrome.searchBox.oncancel";
+static const char kDispatchCancelEventScript[] =
+    "if (window.chrome &&"
+    "    window.chrome.searchBox &&"
+    "    window.chrome.searchBox.oncancel &&"
+    "    typeof window.chrome.searchBox.oncancel == 'function') {"
+    "  window.chrome.searchBox.oncancel();"
+    "  true;"
+    "}";
 
-static const char kResizeEventName[] = "chrome.searchBox.onresize";
+static const char kDispatchResizeEventScript[] =
+    "if (window.chrome &&"
+    "    window.chrome.searchBox &&"
+    "    window.chrome.searchBox.onresize &&"
+    "    typeof window.chrome.searchBox.onresize == 'function') {"
+    "  window.chrome.searchBox.onresize();"
+    "  true;"
+    "}";
 
 // Deprecated API support.
 // TODO(tonyg): Remove these when they are no longer used.
 // ----------------------------------------------------------------------------
 // Script sent as the user is typing and the provider supports instant.
 // Params:
 // . the text the user typed.
 // '46' forces the server to give us verbatim results.
 static const char kUserInputScript[] =
     "if (window.chrome.userInput)"
@@ skipping 263 matching lines @@
       }
     }
   }
 
   if (content::RenderView* render_view = GetRenderView())
     SearchBox::Get(render_view)->SetSuggestions(suggestions, behavior);
   return v8::Undefined();
 }
 
 // static
-bool Dispatch(WebFrame* frame, const std::string& event_name) {
+void Dispatch(WebFrame* frame,
+              WebString event_dispatch_script,
+              WebString no_event_handler_script) {
   DCHECK(frame) << "Dispatch requires frame";
-  if (!frame) return false;
+  if (!frame)
+    return;
 
-  v8::HandleScope handle_scope;
-  v8::Local<v8::Context> context = frame->mainWorldScriptContext();
-  if (context.IsEmpty())
-    return false;
-  v8::Context::Scope context_scope(context);
-
-  v8::Local<v8::Value> value =
-      context->Global()->Get(v8::String::New("window"));
-  std::vector<std::string> components;
-  base::SplitStringDontTrim(event_name, '.', &components);
-  for (size_t i = 0; i < components.size(); ++i) {
-    if (!value.IsEmpty() && value->IsObject())
-      value = value->ToObject()->Get(v8::String::New(components[i].c_str()));
-  }
-  if (value.IsEmpty() || !value->IsFunction())
-    return false;
-
-  v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(value);
-  if (function.IsEmpty())
-    return false;
-
-  function->Call(v8::Object::New(), 0, NULL);
-  return true;
+  v8::Handle<v8::Value> result = frame->executeScriptAndReturnValue(
+      WebScriptSource(event_dispatch_script));
+  if (result.IsEmpty())
+      frame->executeScript(WebScriptSource(no_event_handler_script));

[sky, 2012/04/16 23:37:16]

nit: 2 space indent.

[rafaelw, 2012/04/16 23:41:29]

Done.

 }
 
 // static
 void SearchBoxExtension::DispatchChange(WebFrame* frame) {
-  if (Dispatch(frame, kChangeEventName))
-    return;
-  frame->executeScript(WebScriptSource(kUserInputScript));
+  Dispatch(frame, kDispatchChangeEventScript, kUserInputScript);
 }
 
 // static
 void SearchBoxExtension::DispatchSubmit(WebFrame* frame) {
-  if (Dispatch(frame, kSubmitEventName))
-    return;
-  frame->executeScript(WebScriptSource(kUserDoneScript));
+  Dispatch(frame, kDispatchSubmitEventScript, kUserDoneScript);
 }
 
 // static
 void SearchBoxExtension::DispatchCancel(WebFrame* frame) {
-  if (Dispatch(frame, kCancelEventName))
-    return;
-  frame->executeScript(WebScriptSource(kUserDoneScript));
+  Dispatch(frame, kDispatchCancelEventScript, kUserDoneScript);
 }
 
 // static
 void SearchBoxExtension::DispatchResize(WebFrame* frame) {
-  if (Dispatch(frame, kResizeEventName))
-    return;
-  frame->executeScript(WebScriptSource(kSetOmniboxBoundsScript));
+  Dispatch(frame, kDispatchResizeEventScript, kSetOmniboxBoundsScript);
 }
 
 // static
 bool SearchBoxExtension::PageSupportsInstant(WebFrame* frame) {
   DCHECK(frame) << "PageSupportsInstant requires frame";
   if (!frame) return false;
 
   v8::Handle<v8::Value> v = frame->executeScriptAndReturnValue(
       WebScriptSource(kSupportsInstantScript));
   bool supports_deprecated_api = !v.IsEmpty() && v->BooleanValue();
@@ skipping 11 matching lines @@
 
   return supports_searchbox_api || supports_deprecated_api;
 }
 
 // static
 v8::Extension* SearchBoxExtension::Get() {
   return new SearchBoxExtensionWrapper();
 }
 
 }  // namespace extensions_v8