From c9f1234b1982eb7ec8a5254195574e88bee54703 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Tue, 24 Jul 2018 16:29:25 +0000 Subject: 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 Commit-Queue: Tom Sepez --- fxjs/cfxjse_context.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'fxjs/cfxjse_context.h') diff --git a/fxjs/cfxjse_context.h b/fxjs/cfxjse_context.h index e0e5eeba55..5a299428ab 100644 --- a/fxjs/cfxjse_context.h +++ b/fxjs/cfxjse_context.h @@ -51,7 +51,6 @@ class CFXJSE_Context { void FXJSE_UpdateObjectBinding(v8::Local& hObject, CFXJSE_HostObject* lpNewBinding); -CFXJSE_HostObject* FXJSE_RetrieveObjectBinding(v8::Local hJSObject, - CFXJSE_Class* lpClass); +CFXJSE_HostObject* FXJSE_RetrieveObjectBinding(v8::Local hJSObject); #endif // FXJS_CFXJSE_CONTEXT_H_ -- cgit v1.2.3