diff options
author | Jochen Eisinger <jochen@chromium.org> | 2016-07-15 13:30:58 +0200 |
---|---|---|
committer | Jochen Eisinger <jochen@chromium.org> | 2016-07-15 13:30:58 +0200 |
commit | e5effaf2556046b01265484922e0b8681790323b (patch) | |
tree | eb710ea53a2294165fcb21096305c241f15d0b5f /xfa/fxjse | |
parent | 4707a8f8ab98e3091e109cab71dea69a27c217db (diff) | |
download | pdfium-e5effaf2556046b01265484922e0b8681790323b.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/2144333003 .
Diffstat (limited to 'xfa/fxjse')
-rw-r--r-- | xfa/fxjse/class.cpp | 25 |
1 files changed, 15 insertions, 10 deletions
diff --git a/xfa/fxjse/class.cpp b/xfa/fxjse/class.cpp index d6744a6112..b3862076c8 100644 --- a/xfa/fxjse/class.cpp +++ b/xfa/fxjse/class.cpp @@ -127,6 +127,9 @@ static void FXJSE_V8SetterCallback_Wrapper( static void FXJSE_V8ConstructorCallback_Wrapper( const v8::FunctionCallbackInfo<v8::Value>& info) { + if (!info.IsConstructCall()) { + return; + } const FXJSE_CLASS* lpClassDefinition = static_cast<FXJSE_CLASS*>(info.Data().As<v8::External>()->Value()); if (!lpClassDefinition) { @@ -267,12 +270,14 @@ CFXJSE_Class* CFXJSE_Class::Create(CFXJSE_Context* lpContext, } if (lpClassDefinition->methNum) { for (int32_t i = 0; i < lpClassDefinition->methNum; i++) { + v8::Local<v8::FunctionTemplate> fun = v8::FunctionTemplate::New( + pIsolate, FXJSE_V8FunctionCallback_Wrapper, + v8::External::New(pIsolate, const_cast<FXJSE_FUNCTION*>( + lpClassDefinition->methods + i))); + fun->RemovePrototype(); hObjectTemplate->Set( v8::String::NewFromUtf8(pIsolate, lpClassDefinition->methods[i].name), - v8::FunctionTemplate::New( - pIsolate, FXJSE_V8FunctionCallback_Wrapper, - v8::External::New(pIsolate, const_cast<FXJSE_FUNCTION*>( - lpClassDefinition->methods + i))), + fun, static_cast<v8::PropertyAttribute>(v8::ReadOnly | v8::DontDelete)); } } @@ -297,12 +302,12 @@ CFXJSE_Class* CFXJSE_Class::Create(CFXJSE_Context* lpContext, } } if (bIsJSGlobal) { - hObjectTemplate->Set( - v8::String::NewFromUtf8(pIsolate, "toString"), - v8::FunctionTemplate::New( - pIsolate, FXJSE_Context_GlobalObjToString, - v8::External::New(pIsolate, - const_cast<FXJSE_CLASS*>(lpClassDefinition)))); + v8::Local<v8::FunctionTemplate> fun = v8::FunctionTemplate::New( + pIsolate, FXJSE_Context_GlobalObjToString, + v8::External::New(pIsolate, + const_cast<FXJSE_CLASS*>(lpClassDefinition))); + fun->RemovePrototype(); + hObjectTemplate->Set(v8::String::NewFromUtf8(pIsolate, "toString"), fun); } pClass->m_hTemplate.Reset(lpContext->m_pIsolate, hFunctionTemplate); lpContext->m_rgClasses.Add(pClass); |