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/Icon.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/Icon.h')
| -rw-r--r-- | fpdfsdk/javascript/Icon.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/fpdfsdk/javascript/Icon.h b/fpdfsdk/javascript/Icon.h index e856ee9e51..df2e4bdcc3 100644 --- a/fpdfsdk/javascript/Icon.h +++ b/fpdfsdk/javascript/Icon.h @@ -16,7 +16,11 @@ class Icon : public CJS_EmbedObj { explicit Icon(CJS_Object* pJSObject); ~Icon() override; - bool name(CJS_Runtime* pRuntime, CJS_PropValue& vp, WideString& sError); + bool get_name(CJS_Runtime* pRuntime, CJS_PropValue* vp, WideString* sError); + bool set_name(CJS_Runtime* pRuntime, + const CJS_PropValue& vp, + WideString* sError); + WideString GetIconName() const { return m_swIconName; } void SetIconName(WideString name) { m_swIconName = name; } @@ -30,7 +34,7 @@ class CJS_Icon : public CJS_Object { ~CJS_Icon() override {} DECLARE_JS_CLASS(); - JS_STATIC_PROP(name, Icon); + JS_STATIC_PROP(name, name, Icon); }; #endif // FPDFSDK_JAVASCRIPT_ICON_H_ |