diff options
author | Jochen Eisinger <jochen@chromium.org> | 2016-07-15 10:11:01 +0200 |
---|---|---|
committer | Jochen Eisinger <jochen@chromium.org> | 2016-07-15 10:11:01 +0200 |
commit | 460df6050fe969f30d8c1fce80689303ac02c674 (patch) | |
tree | 9064bbdd2252b47d23905d21e6cef01be00778cc /fxjse | |
parent | cc13c896b8dc7438673d3030a24cfb26c50accb1 (diff) | |
download | pdfium-460df6050fe969f30d8c1fce80689303ac02c674.tar.xz |
Remove constructor from functions that aren't constructors
BUG=chromium:625823
TBR=haraken@chromium.org,thestig@chromium.org
Review-Url: https://codereview.chromium.org/2128793002
(cherry picked from commit 3c27a84d15c06f85cc7f455f96dc124673f9f9d2)
Review URL: https://codereview.chromium.org/2149763005 .
Diffstat (limited to 'fxjse')
-rw-r--r-- | fxjse/dynprop.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/fxjse/dynprop.cpp b/fxjse/dynprop.cpp index 34f63f4338..836cd5f25a 100644 --- a/fxjse/dynprop.cpp +++ b/fxjse/dynprop.cpp @@ -57,9 +57,11 @@ static void FXJSE_DynPropGetterAdapter(const FXJSE_CLASS_DESCRIPTOR* lpClass, 1, v8::String::NewFromUtf8( pIsolate, reinterpret_cast<const char*>(szPropName.raw_str()), v8::String::kNormalString, szPropName.GetLength())); - pValue->ForceSetValue(v8::Function::New( - pValue->GetIsolate(), FXJSE_DynPropGetterAdapter_MethodCallback, - hCallBackInfo)); + pValue->ForceSetValue( + v8::Function::New(pValue->GetIsolate()->GetCurrentContext(), + FXJSE_DynPropGetterAdapter_MethodCallback, + hCallBackInfo, 0, v8::ConstructorBehavior::kThrow) + .ToLocalChecked()); } } } |