diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-07-24 16:29:25 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-07-24 16:29:25 +0000 |
| commit | c9f1234b1982eb7ec8a5254195574e88bee54703 (patch) | |
| tree | cdc8831de3fae8d571646c29bad3da80cd63799c /fxjs/cfxjse_context.h | |
| parent | aa50728980036f07fda232cea974fd80c89b7cb7 (diff) | |
| download | pdfium-c9f1234b1982eb7ec8a5254195574e88bee54703.tar.xz | |
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>
Diffstat (limited to 'fxjs/cfxjse_context.h')
| -rw-r--r-- | fxjs/cfxjse_context.h | 3 |
1 files changed, 1 insertions, 2 deletions
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<v8::Object>& hObject, CFXJSE_HostObject* lpNewBinding); -CFXJSE_HostObject* FXJSE_RetrieveObjectBinding(v8::Local<v8::Object> hJSObject, - CFXJSE_Class* lpClass); +CFXJSE_HostObject* FXJSE_RetrieveObjectBinding(v8::Local<v8::Object> hJSObject); #endif // FXJS_CFXJSE_CONTEXT_H_ |