diff options
| author | dan sinclair <dsinclair@chromium.org> | 2017-10-19 14:29:33 -0400 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-10-19 19:50:07 +0000 |
| commit | cbe23dbdff3f6e41843fd99cbf615000b52727ed (patch) | |
| tree | f2f9e6e5f4dc8818ef15a8f72fdfdab71a848097 /fpdfsdk/javascript/global.h | |
| parent | c136b3146257d0f12d1347a9f1a4784372e19a56 (diff) | |
| download | pdfium-cbe23dbdff3f6e41843fd99cbf615000b52727ed.tar.xz | |
Refactoring JS Callbacks.
This CL updates the fpdfsdk/javascript callbacks to have explicit
get/set methods instead of one method which worked differently
depending on the mode.
This allows better ownership of the passed in params, (get takes a *
and set takes a const&). The Value object was changed to have To*
and Set methods to make the code clearer compared to the operator<<
and operator>> overloading.
Bug:
Change-Id: Id6ff20a4e3252adfd0a78b643e50b9f095085018
Reviewed-on: https://pdfium-review.googlesource.com/16330
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fpdfsdk/javascript/global.h')
| -rw-r--r-- | fpdfsdk/javascript/global.h | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/fpdfsdk/javascript/global.h b/fpdfsdk/javascript/global.h index 5005b2aeed..c741a1a20f 100644 --- a/fpdfsdk/javascript/global.h +++ b/fpdfsdk/javascript/global.h @@ -41,13 +41,13 @@ class JSGlobalAlternate : public CJS_EmbedObj { CJS_Value& vRet, WideString& sError); bool QueryProperty(const wchar_t* propname); - bool DoProperty(CJS_Runtime* pRuntime, - const wchar_t* propname, - CJS_PropValue& vp, - WideString& sError); - bool DelProperty(CJS_Runtime* pRuntime, + bool GetProperty(CJS_Runtime* pRuntime, const wchar_t* propname, - WideString& sError); + CJS_PropValue* vp); + bool SetProperty(CJS_Runtime* pRuntime, + const wchar_t* propname, + const CJS_PropValue& vp); + bool DelProperty(CJS_Runtime* pRuntime, const wchar_t* propname); void Initial(CPDFSDK_FormFillEnvironment* pFormFillEnv); private: |