Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(13)

Side by Side Diff: chrome/renderer/external_extension.cc

Issue 164225: Switch to WebFrame from the WebKit API.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
Property Changes:
Added: svn:eol-style
+ LF
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/renderer/external_extension.h" 5 #include "chrome/renderer/external_extension.h"
6 #include "chrome/renderer/render_view.h" 6 #include "chrome/renderer/render_view.h"
7 #include "webkit/glue/webframe.h" 7 #include "webkit/api/public/WebFrame.h"
8
9 using WebKit::WebFrame;
8 10
9 namespace extensions_v8 { 11 namespace extensions_v8 {
10 12
11 const char* kExternalExtensionName = "v8/External"; 13 const char* kExternalExtensionName = "v8/External";
12 14
13 class ExternalExtensionWrapper : public v8::Extension { 15 class ExternalExtensionWrapper : public v8::Extension {
14 public: 16 public:
15 ExternalExtensionWrapper() 17 ExternalExtensionWrapper()
16 : v8::Extension( 18 : v8::Extension(
17 kExternalExtensionName, 19 kExternalExtensionName,
(...skipping 10 matching lines...) Expand all
28 if (name->Equals(v8::String::New("NativeAddSearchProvider"))) { 30 if (name->Equals(v8::String::New("NativeAddSearchProvider"))) {
29 return v8::FunctionTemplate::New(AddSearchProvider); 31 return v8::FunctionTemplate::New(AddSearchProvider);
30 } 32 }
31 return v8::Handle<v8::FunctionTemplate>(); 33 return v8::Handle<v8::FunctionTemplate>();
32 } 34 }
33 35
34 static v8::Handle<v8::Value> AddSearchProvider(const v8::Arguments& args) { 36 static v8::Handle<v8::Value> AddSearchProvider(const v8::Arguments& args) {
35 if (!args.Length()) 37 if (!args.Length())
36 return v8::Undefined(); 38 return v8::Undefined();
37 39
38 WebFrame* webframe = WebFrame::RetrieveFrameForEnteredContext(); 40 WebFrame* webframe = WebFrame::frameForEnteredContext();
39 DCHECK(webframe) << "There should be an active frame since we just got " 41 DCHECK(webframe) << "There should be an active frame since we just got "
40 "a native function called."; 42 "a native function called.";
41 if (!webframe) return v8::Undefined(); 43 if (!webframe) return v8::Undefined();
42 44
43 WebView* webview = webframe->GetView(); 45 WebView* webview = webframe->view();
44 if (!webview) return v8::Undefined(); // can happen during closing 46 if (!webview) return v8::Undefined(); // can happen during closing
45 47
46 RenderView* renderview = static_cast<RenderView*>(webview->GetDelegate()); 48 RenderView* renderview = static_cast<RenderView*>(webview->GetDelegate());
47 if (!renderview) return v8::Undefined(); 49 if (!renderview) return v8::Undefined();
48 50
49 std::string name = std::string(*v8::String::Utf8Value(args[0])); 51 std::string name = std::string(*v8::String::Utf8Value(args[0]));
50 if (!name.length()) return v8::Undefined();; 52 if (!name.length()) return v8::Undefined();;
51 53
52 renderview->AddSearchProvider(name); 54 renderview->AddSearchProvider(name);
53 return v8::Undefined(); 55 return v8::Undefined();
54 } 56 }
55 }; 57 };
56 58
57 v8::Extension* ExternalExtension::Get() { 59 v8::Extension* ExternalExtension::Get() {
58 return new ExternalExtensionWrapper(); 60 return new ExternalExtensionWrapper();
59 } 61 }
60 62
61 } // namespace extensions_v8 63 } // namespace extensions_v8
OLDNEW
« no previous file with comments | « chrome/renderer/extensions/extension_process_bindings.cc ('k') | chrome/renderer/external_host_bindings.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698