From 3c27a84d15c06f85cc7f455f96dc124673f9f9d2 Mon Sep 17 00:00:00 2001
From: jochen <jochen@chromium.org>
Date: Thu, 7 Jul 2016 04:31:26 -0700
Subject: Remove constructor from functions that aren't constructors

BUG=chromium:625823
R=haraken@chromium.org,thestig@chromium.org

Review-Url: https://codereview.chromium.org/2128793002
---
 fxjse/dynprop.cpp | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

(limited to 'fxjse')

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());
     }
   }
 }
-- 
cgit v1.2.3