diff options
author | Lei Zhang <thestig@chromium.org> | 2018-10-10 18:44:45 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-10-10 18:44:45 +0000 |
commit | 4f261fffddddd97602e88fe462fef1a18a0b4782 (patch) | |
tree | e3a93a859d189a5fe46791d7954396b2b394cbd4 /fpdfsdk/cpdfsdk_widget.h | |
parent | b353f8c32b2d17cbea5ace6eb5f7edb8663e37be (diff) | |
download | pdfium-4f261fffddddd97602e88fe462fef1a18a0b4782.tar.xz |
Get rid of non-const ref parameter in CPDFSDK_InterForm::OnFormat().
Return Optional<WideString> instead. Change the call stack to accept
Optional<WideString>.
Change-Id: I020589fac8a1de4974070268a19b4acad64c4092
Reviewed-on: https://pdfium-review.googlesource.com/c/43601
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fpdfsdk/cpdfsdk_widget.h')
-rw-r--r-- | fpdfsdk/cpdfsdk_widget.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fpdfsdk/cpdfsdk_widget.h b/fpdfsdk/cpdfsdk_widget.h index fb83ceb555..e3f9204eb8 100644 --- a/fpdfsdk/cpdfsdk_widget.h +++ b/fpdfsdk/cpdfsdk_widget.h @@ -89,10 +89,10 @@ class CPDFSDK_Widget final : public CPDFSDK_BAAnnot { #ifdef PDF_ENABLE_XFA void ResetAppearance(bool bValueChanged); #endif // PDF_ENABLE_XFA - void ResetAppearance(const WideString* sValue, bool bValueChanged); + void ResetAppearance(Optional<WideString> sValue, bool bValueChanged); void ResetFieldAppearance(bool bValueChanged); void UpdateField(); - WideString OnFormat(bool& bFormatted); + Optional<WideString> OnFormat(); bool OnAAction(CPDF_AAction::AActionType type, CPDFSDK_FieldAction* data, |