diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-21 21:59:45 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-21 21:59:45 +0000 |
| commit | 96d6f741b3c106ba0e48a05692910f64cf1e5e09 (patch) | |
| tree | 07fc97f427a308bffd995ae9a77aa2dbac9972eb /xfa/fxfa/cxfa_textparser.cpp | |
| parent | b06d3f8d7658eeab6624f66aef690a7f4b44936e (diff) | |
| download | pdfium-96d6f741b3c106ba0e48a05692910f64cf1e5e09.tar.xz | |
Cleanup CXFA_ParaData
This CL fixes the return types and makes the CXFA_ParaData methods
const.
Change-Id: Ifb63b46e826d9741682d992d7fc1ee37c772fb41
Reviewed-on: https://pdfium-review.googlesource.com/19070
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_textparser.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_textparser.cpp | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/xfa/fxfa/cxfa_textparser.cpp b/xfa/fxfa/cxfa_textparser.cpp index 8981799d83..433531373c 100644 --- a/xfa/fxfa/cxfa_textparser.cpp +++ b/xfa/fxfa/cxfa_textparser.cpp @@ -115,6 +115,12 @@ RetainPtr<CFX_CSSComputedStyle> CXFA_TextParser::CreateRootStyle( case XFA_ATTRIBUTEENUM_JustifyAll: hAlign = CFX_CSSTextAlign::JustifyAll; break; + case XFA_ATTRIBUTEENUM_Left: + case XFA_ATTRIBUTEENUM_Radix: + break; + default: + NOTREACHED(); + break; } pStyle->SetTextAlign(hAlign); CFX_CSSRect rtMarginWidth; @@ -291,7 +297,8 @@ std::unique_ptr<CXFA_TextParser::TagProvider> CXFA_TextParser::ParseTagInfo( return tagProvider; } -int32_t CXFA_TextParser::GetVAlign(CXFA_TextProvider* pTextProvider) const { +XFA_ATTRIBUTEENUM CXFA_TextParser::GetVAlign( + CXFA_TextProvider* pTextProvider) const { CXFA_ParaData paraData = pTextProvider->GetParaData(); return paraData.HasValidNode() ? paraData.GetVerticalAlign() : XFA_ATTRIBUTEENUM_Top; |