summaryrefslogtreecommitdiff
path: root/fxjs/xfa
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 /fxjs/xfa
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 'fxjs/xfa')
-rw-r--r--fxjs/xfa/cjx_field.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/fxjs/xfa/cjx_field.cpp b/fxjs/xfa/cjx_field.cpp
index 0007bad592..299c6ef3a4 100644
--- a/fxjs/xfa/cjx_field.cpp
+++ b/fxjs/xfa/cjx_field.cpp
@@ -272,7 +272,8 @@ void CJX_Field::defaultValue(CFXJSE_Value* pValue,
}
CXFA_Node* pUIChild = pWidgetAcc->GetUIChild();
- CXFA_Node* pNode = xfaNode->GetFormValue()->GetFirstChild();
+ CXFA_Node* formValue = xfaNode->GetFormValueIfExists();
+ CXFA_Node* pNode = formValue ? formValue->GetFirstChild() : nullptr;
if (pNode && pNode->GetElementType() == XFA_Element::Decimal) {
if (pUIChild->GetElementType() == XFA_Element::NumericEdit &&
(pNode->JSObject()->GetInteger(XFA_Attribute::FracDigits) == -1)) {