diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-06-22 17:46:05 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-22 17:46:05 +0000 |
| commit | 9fff1c76c1083a5cc0543911811e36007a38aa3a (patch) | |
| tree | d7769c918bdf41a572d3869c5f2e3982636b50a0 /fxjs/cfxjse_engine.h | |
| parent | e6ca7c2d7d45bd59d53c3c6f9952e2001cf81890 (diff) | |
| download | pdfium-9fff1c76c1083a5cc0543911811e36007a38aa3a.tar.xz | |
Avoid some duplicate const/non-const getters.
Getters should nearly always be const when they return a
non-const pointer to an unowned object. Saves a bit of
code along the way.
Grep for similar occurrences and add "const".
Change-Id: I492bf962a7d62452fa40310146226dc0c8ebb753
Reviewed-on: https://pdfium-review.googlesource.com/35890
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'fxjs/cfxjse_engine.h')
| -rw-r--r-- | fxjs/cfxjse_engine.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fxjs/cfxjse_engine.h b/fxjs/cfxjse_engine.h index 62de0614c4..0ae16df84e 100644 --- a/fxjs/cfxjse_engine.h +++ b/fxjs/cfxjse_engine.h @@ -56,7 +56,7 @@ class CFXJSE_Engine : public CFX_V8 { ~CFXJSE_Engine() override; void SetEventParam(CXFA_EventParam* param) { m_eventParam = param; } - CXFA_EventParam* GetEventParam() { return m_eventParam.Get(); } + CXFA_EventParam* GetEventParam() const { return m_eventParam.Get(); } bool RunScript(CXFA_Script::Type eScriptType, const WideStringView& wsScript, CFXJSE_Value* pRetValue, |