diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-04 01:49:37 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-04 01:49:37 +0000 |
| commit | c5d5729700a4d145cd298ec52f6f57b2d11aaafe (patch) | |
| tree | b9929296fc45c754d5f7748613cec15020664052 /xfa/fxfa/parser/cxfa_caption.cpp | |
| parent | cf705715c5311057933ac03b395d9ea75b6f0660 (diff) | |
| download | pdfium-c5d5729700a4d145cd298ec52f6f57b2d11aaafe.tar.xz | |
Remove default values from CXFA_Node::{CountChildren|GetChild}chromium/3259
This CL removes the default values and inlines into the call sites as
needed.
Change-Id: I554e9d5ac781003695f90522f9c6c3160653eb59
Reviewed-on: https://pdfium-review.googlesource.com/17572
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_caption.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_caption.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/xfa/fxfa/parser/cxfa_caption.cpp b/xfa/fxfa/parser/cxfa_caption.cpp index ccfe606b33..3f820e437e 100644 --- a/xfa/fxfa/parser/cxfa_caption.cpp +++ b/xfa/fxfa/parser/cxfa_caption.cpp @@ -30,15 +30,16 @@ float CXFA_Caption::GetReserve() { } CXFA_Margin CXFA_Caption::GetMargin() { - return CXFA_Margin(m_pNode ? m_pNode->GetChild(0, XFA_Element::Margin) + return CXFA_Margin(m_pNode ? m_pNode->GetChild(0, XFA_Element::Margin, false) : nullptr); } CXFA_Font CXFA_Caption::GetFont() { - return CXFA_Font(m_pNode ? m_pNode->GetChild(0, XFA_Element::Font) : nullptr); + return CXFA_Font(m_pNode ? m_pNode->GetChild(0, XFA_Element::Font, false) + : nullptr); } CXFA_Value CXFA_Caption::GetValue() { - return CXFA_Value(m_pNode ? m_pNode->GetChild(0, XFA_Element::Value) + return CXFA_Value(m_pNode ? m_pNode->GetChild(0, XFA_Element::Value, false) : nullptr); } |