summaryrefslogtreecommitdiff
path: root/xfa/fxfa/parser
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2018-01-11 14:34:56 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-01-11 14:34:56 +0000
commit19835921655cd6f2086c7a741696906fc340abac (patch)
tree7b71d4d2ae2985a9418160e41e47eeea4eff370e /xfa/fxfa/parser
parent241752f3c188ed992a26618361e55b1d7f2be7cc (diff)
downloadpdfium-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.cpp2
-rw-r--r--xfa/fxfa/parser/cxfa_node.h2
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();