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>

2 files changed, 16 insertions, 16 deletions

diff --git a/xfa/fxfa/parser/cxfa_paradata.cpp b/xfa/fxfa/parser/cxfa_paradata.cpp
index 95dc80a934..cb960f8dca 100644
--- a/xfa/fxfa/parser/cxfa_paradata.cpp
+++ b/xfa/fxfa/parser/cxfa_paradata.cpp
@@ -11,49 +11,49 @@
 
 CXFA_ParaData::CXFA_ParaData(CXFA_Node* pNode) : CXFA_DataData(pNode) {}
 
-int32_t CXFA_ParaData::GetHorizontalAlign() {
+XFA_ATTRIBUTEENUM CXFA_ParaData::GetHorizontalAlign() const {
   return m_pNode->JSNode()
       ->TryEnum(XFA_Attribute::HAlign, true)
       .value_or(XFA_ATTRIBUTEENUM_Left);
 }
 
-int32_t CXFA_ParaData::GetVerticalAlign() {
+XFA_ATTRIBUTEENUM CXFA_ParaData::GetVerticalAlign() const {
   return m_pNode->JSNode()
       ->TryEnum(XFA_Attribute::VAlign, true)
       .value_or(XFA_ATTRIBUTEENUM_Top);
 }
 
-float CXFA_ParaData::GetLineHeight() {
+float CXFA_ParaData::GetLineHeight() const {
   return m_pNode->JSNode()
       ->GetMeasure(XFA_Attribute::LineHeight)
       .ToUnit(XFA_Unit::Pt);
 }
 
-float CXFA_ParaData::GetMarginLeft() {
+float CXFA_ParaData::GetMarginLeft() const {
   return m_pNode->JSNode()
       ->GetMeasure(XFA_Attribute::MarginLeft)
       .ToUnit(XFA_Unit::Pt);
 }
 
-float CXFA_ParaData::GetMarginRight() {
+float CXFA_ParaData::GetMarginRight() const {
   return m_pNode->JSNode()
       ->GetMeasure(XFA_Attribute::MarginRight)
       .ToUnit(XFA_Unit::Pt);
 }
 
-float CXFA_ParaData::GetSpaceAbove() {
+float CXFA_ParaData::GetSpaceAbove() const {
   return m_pNode->JSNode()
       ->GetMeasure(XFA_Attribute::SpaceAbove)
       .ToUnit(XFA_Unit::Pt);
 }
 
-float CXFA_ParaData::GetSpaceBelow() {
+float CXFA_ParaData::GetSpaceBelow() const {
   return m_pNode->JSNode()
       ->GetMeasure(XFA_Attribute::SpaceBelow)
       .ToUnit(XFA_Unit::Pt);
 }
 
-float CXFA_ParaData::GetTextIndent() {
+float CXFA_ParaData::GetTextIndent() const {
   return m_pNode->JSNode()
       ->GetMeasure(XFA_Attribute::TextIndent)
       .ToUnit(XFA_Unit::Pt);
diff --git a/xfa/fxfa/parser/cxfa_paradata.h b/xfa/fxfa/parser/cxfa_paradata.h
index 20c371ed51..b08c5c471f 100644
--- a/xfa/fxfa/parser/cxfa_paradata.h
+++ b/xfa/fxfa/parser/cxfa_paradata.h
@@ -16,14 +16,14 @@ class CXFA_ParaData : public CXFA_DataData {
  public:
   explicit CXFA_ParaData(CXFA_Node* pNode);
 
-  int32_t GetHorizontalAlign();
-  int32_t GetVerticalAlign();
-  float GetLineHeight();
-  float GetMarginLeft();
-  float GetMarginRight();
-  float GetSpaceAbove();
-  float GetSpaceBelow();
-  float GetTextIndent();
+  XFA_ATTRIBUTEENUM GetHorizontalAlign() const;
+  XFA_ATTRIBUTEENUM GetVerticalAlign() const;
+  float GetLineHeight() const;
+  float GetMarginLeft() const;
+  float GetMarginRight() const;
+  float GetSpaceAbove() const;
+  float GetSpaceBelow() const;
+  float GetTextIndent() const;
 };
 
 #endif  // XFA_FXFA_PARSER_CXFA_PARADATA_H_