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_value.cpp | |
| 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_value.cpp')
| -rw-r--r-- | fxjs/cfxjse_value.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fxjs/cfxjse_value.cpp b/fxjs/cfxjse_value.cpp index 915fefc119..90f553b169 100644 --- a/fxjs/cfxjse_value.cpp +++ b/fxjs/cfxjse_value.cpp @@ -76,7 +76,7 @@ CFXJSE_HostObject* CFXJSE_Value::ToHostObject() const { if (!pValue->IsObject()) return nullptr; - return FXJSE_RetrieveObjectBinding(pValue.As<v8::Object>(), nullptr); + return FXJSE_RetrieveObjectBinding(pValue.As<v8::Object>()); } void CFXJSE_Value::SetObject(CFXJSE_HostObject* lpObject, |