diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-11 14:34:56 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-11 14:34:56 +0000 |
| commit | 19835921655cd6f2086c7a741696906fc340abac (patch) | |
| tree | 7b71d4d2ae2985a9418160e41e47eeea4eff370e /xfa/fxfa/parser | |
| parent | 241752f3c188ed992a26618361e55b1d7f2be7cc (diff) | |
| download | pdfium-19835921655cd6f2086c7a741696906fc340abac.tar.xz | |
Rename GetFormValue to GetFormValueIfExists
This CL makes it clear that GetFormValue can return nullptr and fixes up
a few callsites as needed.
Change-Id: Id2e85b0956a678b5b338b9e2a4d32731fe6c4729
Reviewed-on: https://pdfium-review.googlesource.com/22690
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_node.cpp | 2 | ||||
| -rw-r--r-- | xfa/fxfa/parser/cxfa_node.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fxfa/parser/cxfa_node.cpp b/xfa/fxfa/parser/cxfa_node.cpp index ed2093cb7a..9c28668c16 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -1600,7 +1600,7 @@ CXFA_Value* CXFA_Node::GetDefaultValueIfExists() { : nullptr; } -CXFA_Value* CXFA_Node::GetFormValue() const { +CXFA_Value* CXFA_Node::GetFormValueIfExists() const { return JSObject()->GetProperty<CXFA_Value>(0, XFA_Element::Value); } diff --git a/xfa/fxfa/parser/cxfa_node.h b/xfa/fxfa/parser/cxfa_node.h index 7aea742a96..8a0a3610a6 100644 --- a/xfa/fxfa/parser/cxfa_node.h +++ b/xfa/fxfa/parser/cxfa_node.h @@ -253,7 +253,7 @@ class CXFA_Node : public CXFA_Object { CXFA_Validate* GetOrCreateValidateIfPossible(); CXFA_Value* GetDefaultValueIfExists(); - CXFA_Value* GetFormValue() const; + CXFA_Value* GetFormValueIfExists() const; WideString GetRawValue(); int32_t GetRotate(); |