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

Side by Side Diff: chrome/renderer/extensions/pepper_request_natives.cc

Issue 234413005: Move most of ChromeV8Context to a base ScriptContext (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/extensions/pepper_request_natives.h" 5 #include "chrome/renderer/extensions/pepper_request_natives.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/values.h" 10 #include "base/values.h"
11 #include "chrome/renderer/extensions/chrome_v8_context.h" 11 #include "chrome/renderer/extensions/chrome_v8_context.h"
12 #include "content/public/renderer/v8_value_converter.h" 12 #include "content/public/renderer/v8_value_converter.h"
13 13
14 namespace extensions { 14 namespace extensions {
15 15
16 PepperRequestNatives::PepperRequestNatives(ChromeV8Context* context) 16 PepperRequestNatives::PepperRequestNatives(ChromeV8Context* context)
17 : ObjectBackedNativeHandler(context) { 17 : ObjectBackedNativeHandler(context) {
18 RouteFunction( 18 RouteFunction(
19 "SendResponse", 19 "SendResponse",
20 base::Bind(&PepperRequestNatives::SendResponse, base::Unretained(this))); 20 base::Bind(&PepperRequestNatives::SendResponse, base::Unretained(this)));
21 } 21 }
22 22
23 void PepperRequestNatives::SendResponse( 23 void PepperRequestNatives::SendResponse(
24 const v8::FunctionCallbackInfo<v8::Value>& args) { 24 const v8::FunctionCallbackInfo<v8::Value>& args) {
25 DCHECK_EQ(3, args.Length()); 25 DCHECK_EQ(3, args.Length());
26 DCHECK(args[0]->IsInt32()); 26 DCHECK(args[0]->IsInt32());
27 DCHECK(args[1]->IsArray()); 27 DCHECK(args[1]->IsArray());
28 int request_id = args[0]->Int32Value(); 28 int request_id = args[0]->Int32Value();
29 ChromeV8Context* chrome_context = static_cast<ChromeV8Context*>(context());
not at google - send to devlin 2014/04/11 15:38:55 could you add a note here explaining this? maybe f
Ken Rockot(use gerrit already) 2014/04/11 17:02:18 Done.
29 if (args[2]->IsString()) { 30 if (args[2]->IsString()) {
30 context()->pepper_request_proxy()->OnResponseReceived( 31 chrome_context->pepper_request_proxy()->OnResponseReceived(
31 request_id, false, base::ListValue(), *v8::String::Utf8Value(args[2])); 32 request_id, false, base::ListValue(), *v8::String::Utf8Value(args[2]));
32 return; 33 return;
33 } 34 }
34 35
35 scoped_ptr<content::V8ValueConverter> converter( 36 scoped_ptr<content::V8ValueConverter> converter(
36 content::V8ValueConverter::create()); 37 content::V8ValueConverter::create());
37 scoped_ptr<const base::Value> result( 38 scoped_ptr<const base::Value> result(
38 converter->FromV8Value(args[1], context()->v8_context())); 39 converter->FromV8Value(args[1], chrome_context->v8_context()));
39 DCHECK(result); 40 DCHECK(result);
40 const base::ListValue* result_list = NULL; 41 const base::ListValue* result_list = NULL;
41 CHECK(result->GetAsList(&result_list)); 42 CHECK(result->GetAsList(&result_list));
42 context()->pepper_request_proxy()->OnResponseReceived( 43 chrome_context->pepper_request_proxy()->OnResponseReceived(
43 request_id, true, *result_list, ""); 44 request_id, true, *result_list, "");
44 } 45 }
45 46
46 } // namespace extensions 47 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698