Chromium Code Reviews| Index: core/fpdfdoc/csection.h |
| diff --git a/core/fpdfdoc/csection.h b/core/fpdfdoc/csection.h |
| index d8974cb02010af0e1ec1f92808d16f8adcb7abc1..442cb6d61c0bdb0cea31ff41e25cc3e25b82c708 100644 |
| --- a/core/fpdfdoc/csection.h |
| +++ b/core/fpdfdoc/csection.h |
| @@ -18,10 +18,11 @@ struct CPVT_LineInfo; |
| struct CPVT_WordLine; |
| struct CPVT_WordPlace; |
| -class CSection { |
| +class CSection final { |
| public: |
| explicit CSection(CPDF_VariableText* pVT); |
| - virtual ~CSection(); |
| + ~CSection(); |
| + |
| void ResetAll(); |
| void ResetLineArray(); |
| void ResetWordArray(); |
| @@ -32,7 +33,7 @@ class CSection { |
| void ClearWords(const CPVT_WordRange& PlaceRange); |
| void ClearWord(const CPVT_WordPlace& place); |
| CPVT_FloatRect Rearrange(); |
| - CPVT_Size GetSectionSize(FX_FLOAT fFontSize); |
| + CFX_PointF GetSectionSize(FX_FLOAT fFontSize); |
|
Wei Li
2016/05/10 20:09:42
Is CFX_SizeF better here?
Lei Zhang
2016/05/11 01:25:12
Sure. It turns out the two are actually the same.
|
| CPVT_WordPlace GetBeginWordPlace() const; |
| CPVT_WordPlace GetEndWordPlace() const; |
| CPVT_WordPlace GetPrevWordPlace(const CPVT_WordPlace& place) const; |
| @@ -56,7 +57,7 @@ class CSection { |
| void ClearRightWords(int32_t nWordIndex); |
| void ClearMidWords(int32_t nBeginIndex, int32_t nEndIndex); |
| - CPDF_VariableText* m_pVT; |
| + CPDF_VariableText* const m_pVT; |
| }; |
| #endif // CORE_FPDFDOC_CSECTION_H_ |