Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2014)

Unified Diff: core/fpdfdoc/csection.h

Issue 1961333002: Remove CPVT_Size and CPVT_FloatRange. (Closed) Base URL: https://pdfium.googlesource.com/pdfium@master
Patch Set: Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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_

Powered by Google App Engine
This is Rietveld 408576698