diff options
| author | Jochen Eisinger <jochen@chromium.org> | 2016-07-15 10:07:53 +0200 |
|---|---|---|
| committer | Jochen Eisinger <jochen@chromium.org> | 2016-07-15 10:07:53 +0200 |
| commit | cc13c896b8dc7438673d3030a24cfb26c50accb1 (patch) | |
| tree | 5632118c559b718344d9a226d3b187ac098a8c9d /fpdfsdk | |
| parent | 017052a077d8f900e3098b8f9115078a037ee805 (diff) | |
| download | pdfium-cc13c896b8dc7438673d3030a24cfb26c50accb1.tar.xz | |
Remove prototypes from v8 functions that aren't constructors
BUG=chromium:625823
TBR=haraken@chromium.org,thestig@chromium.org
Review-Url: https://codereview.chromium.org/2123153002
(cherry picked from commit c4dedf32b1f5c71740df5be2a9b1446a01df304c)
Review URL: https://codereview.chromium.org/2148793006 .
Diffstat (limited to 'fpdfsdk')
| -rw-r--r-- | fpdfsdk/jsapi/fxjs_v8.cpp | 33 |
1 files changed, 20 insertions, 13 deletions
diff --git a/fpdfsdk/jsapi/fxjs_v8.cpp b/fpdfsdk/jsapi/fxjs_v8.cpp index 283fac978a..3297b4bd12 100644 --- a/fpdfsdk/jsapi/fxjs_v8.cpp +++ b/fpdfsdk/jsapi/fxjs_v8.cpp @@ -243,13 +243,14 @@ void FXJS_DefineObjMethod(v8::Isolate* pIsolate, CFX_ByteString bsMethodName = CFX_WideString(sMethodName).UTF8Encode(); CFXJS_ObjDefinition* pObjDef = CFXJS_ObjDefinition::ForID(pIsolate, nObjDefnID); + v8::Local<v8::FunctionTemplate> fun = v8::FunctionTemplate::New( + pIsolate, pMethodCall, v8::Local<v8::Value>(), pObjDef->GetSignature()); + fun->RemovePrototype(); pObjDef->GetInstanceTemplate()->Set( v8::String::NewFromUtf8(pIsolate, bsMethodName.c_str(), v8::NewStringType::kNormal) .ToLocalChecked(), - v8::FunctionTemplate::New(pIsolate, pMethodCall, v8::Local<v8::Value>(), - pObjDef->GetSignature()), - v8::ReadOnly); + fun, v8::ReadOnly); } void FXJS_DefineObjProperty(v8::Isolate* pIsolate, @@ -301,11 +302,14 @@ void FXJS_DefineGlobalMethod(v8::Isolate* pIsolate, v8::Isolate::Scope isolate_scope(pIsolate); v8::HandleScope handle_scope(pIsolate); CFX_ByteString bsMethodName = CFX_WideString(sMethodName).UTF8Encode(); - GetGlobalObjectTemplate(pIsolate) - ->Set(v8::String::NewFromUtf8(pIsolate, bsMethodName.c_str(), - v8::NewStringType::kNormal) - .ToLocalChecked(), - v8::FunctionTemplate::New(pIsolate, pMethodCall), v8::ReadOnly); + v8::Local<v8::FunctionTemplate> fun = + v8::FunctionTemplate::New(pIsolate, pMethodCall); + fun->RemovePrototype(); + GetGlobalObjectTemplate(pIsolate)->Set( + v8::String::NewFromUtf8(pIsolate, bsMethodName.c_str(), + v8::NewStringType::kNormal) + .ToLocalChecked(), + fun, v8::ReadOnly); } void FXJS_DefineGlobalConst(v8::Isolate* pIsolate, @@ -314,11 +318,14 @@ void FXJS_DefineGlobalConst(v8::Isolate* pIsolate, v8::Isolate::Scope isolate_scope(pIsolate); v8::HandleScope handle_scope(pIsolate); CFX_ByteString bsConst = CFX_WideString(sConstName).UTF8Encode(); - GetGlobalObjectTemplate(pIsolate) - ->SetAccessorProperty(v8::String::NewFromUtf8(pIsolate, bsConst.c_str(), - v8::NewStringType::kNormal) - .ToLocalChecked(), - v8::FunctionTemplate::New(pIsolate, pConstGetter)); + v8::Local<v8::FunctionTemplate> fun = + v8::FunctionTemplate::New(pIsolate, pConstGetter); + fun->RemovePrototype(); + GetGlobalObjectTemplate(pIsolate)->SetAccessorProperty( + v8::String::NewFromUtf8(pIsolate, bsConst.c_str(), + v8::NewStringType::kNormal) + .ToLocalChecked(), + fun); } void FXJS_InitializeRuntime( |