Remove lpClass argument from FXJSE_RetrieveObjectBinding()

In turn, it too is always nullptr. This shows that the V8
side check for hasInstance() was never being applied. We will
augment this with C++ side checks down the road, since we don't
want to trust V8 anyways.

Change-Id: Iee38f32af9561783dbf253d798bd975029f3a4a2
Reviewed-on: https://pdfium-review.googlesource.com/38594
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

1 files changed, 2 insertions, 2 deletions

diff --git a/fxjs/cfxjse_engine.cpp b/fxjs/cfxjse_engine.cpp
index c5cc89a8d0..e297ed16e2 100644
--- a/fxjs/cfxjse_engine.cpp
+++ b/fxjs/cfxjse_engine.cpp
@@ -81,7 +81,7 @@ CXFA_Object* CFXJSE_Engine::ToObject(
     return nullptr;
 
   CFXJSE_HostObject* pHostObj =
-      FXJSE_RetrieveObjectBinding(info.Holder().As<v8::Object>(), nullptr);
+      FXJSE_RetrieveObjectBinding(info.Holder().As<v8::Object>());
   return pHostObj ? pHostObj->AsCXFAObject() : nullptr;
 }
 
@@ -793,7 +793,7 @@ CXFA_Object* CFXJSE_Engine::ToXFAObject(v8::Local<v8::Value> obj) {
     return nullptr;
 
   CFXJSE_HostObject* pHostObj =
-      FXJSE_RetrieveObjectBinding(obj.As<v8::Object>(), nullptr);
+      FXJSE_RetrieveObjectBinding(obj.As<v8::Object>());
   return pHostObj ? pHostObj->AsCXFAObject() : nullptr;
 }