diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-02 19:17:18 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-02 19:17:18 +0000 |
| commit | 7f571083e35bc4056515f64c7adb7325f1a27b57 (patch) | |
| tree | 6334314440e13f8ffcff27afc0ae7f58de304ea7 /xfa/fxfa/parser/cxfa_font.cpp | |
| parent | 8946aec4c3e727aae806f9858c2329ce368dd517 (diff) | |
| download | pdfium-7f571083e35bc4056515f64c7adb7325f1a27b57.tar.xz | |
Remove default values from CJX_Node::{Set|Try}Enum
This CL removes the default value and inlines into the call sites.
Change-Id: Ib6bc2b27189be3596e83a8b29b7a6a744278ed34
Reviewed-on: https://pdfium-review.googlesource.com/17570
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_font.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_font.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xfa/fxfa/parser/cxfa_font.cpp b/xfa/fxfa/parser/cxfa_font.cpp index d54ac0ba72..209a3c6778 100644 --- a/xfa/fxfa/parser/cxfa_font.cpp +++ b/xfa/fxfa/parser/cxfa_font.cpp @@ -58,7 +58,7 @@ int32_t CXFA_Font::GetUnderline() { int32_t CXFA_Font::GetUnderlinePeriod() { XFA_ATTRIBUTEENUM eAttr = XFA_ATTRIBUTEENUM_All; - m_pNode->JSNode()->TryEnum(XFA_ATTRIBUTE_UnderlinePeriod, eAttr); + m_pNode->JSNode()->TryEnum(XFA_ATTRIBUTE_UnderlinePeriod, eAttr, true); return eAttr; } @@ -74,13 +74,13 @@ void CXFA_Font::GetTypeface(WideStringView& wsTypeFace) { bool CXFA_Font::IsBold() { XFA_ATTRIBUTEENUM eAttr = XFA_ATTRIBUTEENUM_Normal; - m_pNode->JSNode()->TryEnum(XFA_ATTRIBUTE_Weight, eAttr); + m_pNode->JSNode()->TryEnum(XFA_ATTRIBUTE_Weight, eAttr, true); return eAttr == XFA_ATTRIBUTEENUM_Bold; } bool CXFA_Font::IsItalic() { XFA_ATTRIBUTEENUM eAttr = XFA_ATTRIBUTEENUM_Normal; - m_pNode->JSNode()->TryEnum(XFA_ATTRIBUTE_Posture, eAttr); + m_pNode->JSNode()->TryEnum(XFA_ATTRIBUTE_Posture, eAttr, true); return eAttr == XFA_ATTRIBUTEENUM_Italic; } |