summaryrefslogtreecommitdiff
path: root/xfa/fxfa/cxfa_textparser.cpp
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2017-11-21 21:59:45 +0000
committerChromium commit bot <commit-bot@chromium.org>2017-11-21 21:59:45 +0000
commit96d6f741b3c106ba0e48a05692910f64cf1e5e09 (patch)
tree07fc97f427a308bffd995ae9a77aa2dbac9972eb /xfa/fxfa/cxfa_textparser.cpp
parentb06d3f8d7658eeab6624f66aef690a7f4b44936e (diff)
downloadpdfium-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.cpp9
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;