diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-11 14:26:51 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-11 14:26:51 +0000 |
| commit | 188b2e0333d161ffbac7c896f443b886b113b26a (patch) | |
| tree | deceede8e90401a0df24d3ef79edbce4e6d3bad4 /xfa/fxfa/parser | |
| parent | 7981d15a799e3bdfaf6bd3e04fc08a6a43d143ec (diff) | |
| download | pdfium-188b2e0333d161ffbac7c896f443b886b113b26a.tar.xz | |
Rename GetDefaultValue to GetDefaultValueIfExists
This CL makes it explicit that GetDefaultValue can return nullptr.
Change-Id: I871704e291358bc85a554e4f6410953f7223c61d
Reviewed-on: https://pdfium-review.googlesource.com/22683
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 5a08d16986..ed2093cb7a 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -1593,7 +1593,7 @@ bool CXFA_Node::IsOpenAccess() { return true; } -CXFA_Value* CXFA_Node::GetDefaultValue() { +CXFA_Value* CXFA_Node::GetDefaultValueIfExists() { CXFA_Node* pTemNode = GetTemplateNodeIfExists(); return pTemNode ? pTemNode->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 543d895aeb..7aea742a96 100644 --- a/xfa/fxfa/parser/cxfa_node.h +++ b/xfa/fxfa/parser/cxfa_node.h @@ -252,7 +252,7 @@ class CXFA_Node : public CXFA_Object { CXFA_Validate* GetValidateIfExists() const; CXFA_Validate* GetOrCreateValidateIfPossible(); - CXFA_Value* GetDefaultValue(); + CXFA_Value* GetDefaultValueIfExists(); CXFA_Value* GetFormValue() const; WideString GetRawValue(); int32_t GetRotate(); |