diff options
author | mbelshe@google.com <mbelshe@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-01 16:27:42 +0000 |
---|---|---|
committer | mbelshe@google.com <mbelshe@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-01 16:27:42 +0000 |
commit | 3c8e3706b7f42bc0fc380ece4a7965539e8e0d65 (patch) | |
tree | a5a4ca442cac0e9f6e97313e3663700a89b9a988 /chrome/renderer/external_extension.cc | |
parent | 54c89ef568ae71304e7804c75a44fe2f16f859f2 (diff) | |
download | chromium_src-3c8e3706b7f42bc0fc380ece4a7965539e8e0d65.zip chromium_src-3c8e3706b7f42bc0fc380ece4a7965539e8e0d65.tar.gz chromium_src-3c8e3706b7f42bc0fc380ece4a7965539e8e0d65.tar.bz2 |
Replace window.external with a v8::extension.
BUG=http://crbug.com/10957
TEST=Verify that the AddSearchProvider logic still works. This dates back to an old bug http://b/1069781
Review URL: http://codereview.chromium.org/99168
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15056 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/external_extension.cc')
-rw-r--r-- | chrome/renderer/external_extension.cc | 61 |
1 files changed, 61 insertions, 0 deletions
diff --git a/chrome/renderer/external_extension.cc b/chrome/renderer/external_extension.cc new file mode 100644 index 0000000..378e78e --- /dev/null +++ b/chrome/renderer/external_extension.cc @@ -0,0 +1,61 @@ +// Copyright (c) 2009 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/external_extension.h" +#include "chrome/renderer/render_view.h" +#include "webkit/glue/webframe.h" + +namespace extensions_v8 { + +const char* kExternalExtensionName = "v8/External"; + +class ExternalExtensionWrapper : public v8::Extension { + public: + ExternalExtensionWrapper() + : v8::Extension( + kExternalExtensionName, + "var external;" + "if (!external)" + " external = {};" + "external.AddSearchProvider = function(name) {" + " native function NativeAddSearchProvider();" + " NativeAddSearchProvider(name);" + "}") {} + + virtual v8::Handle<v8::FunctionTemplate> GetNativeFunction( + v8::Handle<v8::String> name) { + if (name->Equals(v8::String::New("NativeAddSearchProvider"))) { + return v8::FunctionTemplate::New(AddSearchProvider); + } + return v8::Handle<v8::FunctionTemplate>(); + } + + static v8::Handle<v8::Value> AddSearchProvider(const v8::Arguments& args) { + if (!args.Length()) + return v8::Undefined(); + + WebFrame* webframe = WebFrame::RetrieveActiveFrame(); + DCHECK(webframe) << "There should be an active frame since we just got " + "a native function called."; + if (!webframe) return v8::Undefined(); + + WebView* webview = webframe->GetView(); + if (!webview) return v8::Undefined(); // can happen during closing + + RenderView* renderview = static_cast<RenderView*>(webview->GetDelegate()); + if (!renderview) return v8::Undefined(); + + std::string name = std::string(*v8::String::Utf8Value(args[0])); + if (!name.length()) return v8::Undefined();; + + renderview->AddSearchProvider(name); + return v8::Undefined(); + } +}; + +v8::Extension* ExternalExtension::Get() { + return new ExternalExtensionWrapper(); +} + +} // namespace extensions_v8 |