summaryrefslogtreecommitdiff
path: root/core/include
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2015-06-19 14:43:51 -0700
committerLei Zhang <thestig@chromium.org>2015-06-19 14:43:51 -0700
commitdf558514bf5cb5750d5a7000a843f45f4170d5a4 (patch)
tree8556b5803dd134f86cb526b5ea113258a39cf805 /core/include
parent012ada613a1462f109eb59790f8e5ee91aacdb7e (diff)
downloadpdfium-df558514bf5cb5750d5a7000a843f45f4170d5a4.tar.xz
Small cleanup of CPDF_TextObject and other CPDF_PageObject subclasses.
- Add overrides - Remove empty overrides - Remove dead friends - Cleanup bad formatting R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1199563002.
Diffstat (limited to 'core/include')
-rw-r--r--core/include/fpdfapi/fpdf_pageobj.h153
1 files changed, 78 insertions, 75 deletions
diff --git a/core/include/fpdfapi/fpdf_pageobj.h b/core/include/fpdfapi/fpdf_pageobj.h
index 3414d36fbf..af186ad03b 100644
--- a/core/include/fpdfapi/fpdf_pageobj.h
+++ b/core/include/fpdfapi/fpdf_pageobj.h
@@ -31,13 +31,10 @@ class CPDF_ShadingObject;
class CPDF_FormObject;
class CPDF_InlineImages;
typedef CFX_PathData CPDF_PathData;
+
class CPDF_Path : public CFX_CountRef<CFX_PathData>
{
public:
-
-
-
-
int GetPointCount()
{
return m_pObject->m_PointCount;
@@ -435,7 +432,6 @@ public:
class CPDF_GraphicStates
{
public:
-
void CopyStates(const CPDF_GraphicStates& src);
void DefaultStates();
@@ -450,6 +446,7 @@ public:
CPDF_GeneralState m_GeneralState;
};
+
class CPDF_PageObject : public CPDF_GraphicStates
{
public:
@@ -460,9 +457,7 @@ public:
void Copy(const CPDF_PageObject* pSrcObject);
- virtual void Transform(const CFX_AffineMatrix& matrix) = 0;
-
-
+ virtual void Transform(const CFX_AffineMatrix& matrix) = 0;
void RemoveClipPath();
@@ -474,7 +469,6 @@ public:
void TransformGeneralState(CFX_AffineMatrix& matrix);
-
void SetColorState(CPDF_ColorState state)
{
m_ColorState = state;
@@ -493,91 +487,91 @@ public:
FX_FLOAT m_Bottom;
CPDF_ContentMark m_ContentMark;
-protected:
- virtual void CopyData(const CPDF_PageObject* pSrcObject) {}
+protected:
+ virtual void CopyData(const CPDF_PageObject* pSrcObject) = 0;
void RecalcBBox();
CPDF_PageObject() {}
};
-struct CPDF_TextObjectItem {
-
- FX_DWORD m_CharCode;
-
- FX_FLOAT m_OriginX;
- FX_FLOAT m_OriginY;
+struct CPDF_TextObjectItem {
+ FX_DWORD m_CharCode;
+ FX_FLOAT m_OriginX;
+ FX_FLOAT m_OriginY;
};
+
class CPDF_TextObject : public CPDF_PageObject
{
public:
-
CPDF_TextObject();
+ ~CPDF_TextObject() override;
- virtual ~CPDF_TextObject();
-
- int CountItems() const
+ int CountItems() const
{
return m_nChars;
}
- void GetItemInfo(int index, CPDF_TextObjectItem* pInfo) const;
-
- int CountChars() const;
+ void GetItemInfo(int index, CPDF_TextObjectItem* pInfo) const;
- void GetCharInfo(int index, FX_DWORD& charcode, FX_FLOAT& kerning) const;
- void GetCharInfo(int index, CPDF_TextObjectItem* pInfo) const;
+ int CountChars() const;
- void GetCharRect(int index, CFX_FloatRect& rect) const;
+ void GetCharInfo(int index, FX_DWORD& charcode, FX_FLOAT& kerning) const;
+ void GetCharInfo(int index, CPDF_TextObjectItem* pInfo) const;
+ void GetCharRect(int index, CFX_FloatRect& rect) const;
- FX_FLOAT GetCharWidth(FX_DWORD charcode) const;
- FX_FLOAT GetSpaceCharWidth() const;
+ FX_FLOAT GetCharWidth(FX_DWORD charcode) const;
+ FX_FLOAT GetSpaceCharWidth() const;
- FX_FLOAT GetPosX() const
+ FX_FLOAT GetPosX() const
{
return m_PosX;
}
- FX_FLOAT GetPosY() const
+ FX_FLOAT GetPosY() const
{
return m_PosY;
}
- void GetTextMatrix(CFX_AffineMatrix* pMatrix) const;
+ void GetTextMatrix(CFX_AffineMatrix* pMatrix) const;
- CPDF_Font* GetFont() const
+ CPDF_Font* GetFont() const
{
return m_TextState.GetFont();
}
- FX_FLOAT GetFontSize() const
+ FX_FLOAT GetFontSize() const
{
return m_TextState.GetFontSize();
}
- void SetEmpty();
-
- void SetText(const CFX_ByteString& text);
+ void SetEmpty();
- void SetText(CFX_ByteString* pStrs, FX_FLOAT* pKerning, int nSegs);
+ void SetText(const CFX_ByteString& text);
- void SetText(int nChars, FX_DWORD* pCharCodes, FX_FLOAT* pKernings);
+ void SetText(CFX_ByteString* pStrs, FX_FLOAT* pKerning, int nSegs);
- void SetPosition(FX_FLOAT x, FX_FLOAT y);
+ void SetText(int nChars, FX_DWORD* pCharCodes, FX_FLOAT* pKernings);
- void SetTextState(CPDF_TextState TextState);
- virtual void Transform(const CFX_AffineMatrix& matrix);
+ void SetPosition(FX_FLOAT x, FX_FLOAT y);
- void CalcCharPos(FX_FLOAT* pPosArray) const;
+ void SetTextState(CPDF_TextState TextState);
+ // CPDF_PageObject:
+ void Transform(const CFX_AffineMatrix& matrix) override;
+ void CalcCharPos(FX_FLOAT* pPosArray) const;
- void SetData(int nChars, FX_DWORD* pCharCodes, FX_FLOAT* pCharPos, FX_FLOAT x, FX_FLOAT y);
+ void SetData(int nChars,
+ FX_DWORD* pCharCodes,
+ FX_FLOAT* pCharPos,
+ FX_FLOAT x,
+ FX_FLOAT y);
- void GetData(int& nChars, FX_DWORD*& pCharCodes, FX_FLOAT*& pCharPos)
+ void GetData(int& nChars, FX_DWORD*& pCharCodes, FX_FLOAT*& pCharPos)
{
nChars = m_nChars;
pCharCodes = m_pCharCodes;
@@ -585,44 +579,49 @@ public:
}
- void RecalcPositionData()
+ void RecalcPositionData()
{
- CalcPositionData(NULL, NULL, 1);
+ CalcPositionData(nullptr, nullptr, 1);
}
+
protected:
- virtual void CopyData(const CPDF_PageObject* pSrcObject);
+ friend class CPDF_RenderStatus;
+ friend class CPDF_StreamContentParser;
+ friend class CPDF_TextRenderer;
+ friend class CTextPage;
- FX_FLOAT m_PosX;
+ // CPDF_PageObject:
+ void CopyData(const CPDF_PageObject* pSrcObject) override;
- FX_FLOAT m_PosY;
+ void SetSegments(const CFX_ByteString* pStrs,
+ FX_FLOAT* pKerning,
+ int nSegs);
- int m_nChars;
+ void CalcPositionData(FX_FLOAT* pTextAdvanceX,
+ FX_FLOAT* pTextAdvanceY,
+ FX_FLOAT horz_scale,
+ int level = 0);
- FX_DWORD* m_pCharCodes;
+ FX_FLOAT m_PosX;
+ FX_FLOAT m_PosY;
- FX_FLOAT* m_pCharPos;
+ int m_nChars;
- void SetSegments(const CFX_ByteString* pStrs, FX_FLOAT* pKerning, int nSegs);
+ FX_DWORD* m_pCharCodes;
- void CalcPositionData(FX_FLOAT* pTextAdvanceX, FX_FLOAT* pTextAdvanceY, FX_FLOAT horz_scale, int level = 0);
- friend class CPDF_StreamContentParser;
- friend class CPDF_RenderStatus;
- friend class CPDF_QuickDrawer;
- friend class CPDF_TextRenderer;
- friend class CTextPage;
- friend class CPDF_ContentGenerator;
+ FX_FLOAT* m_pCharPos;
};
+
class CPDF_PathObject : public CPDF_PageObject
{
public:
-
CPDF_PathObject()
{
m_Type = PDFPAGE_PATH;
}
virtual ~CPDF_PathObject() {}
- virtual void Transform(const CFX_AffineMatrix& maxtrix);
+ void Transform(const CFX_AffineMatrix& maxtrix) override;
void SetGraphState(CPDF_GraphState GraphState);
@@ -636,30 +635,32 @@ public:
void CalcBoundingBox();
+
protected:
- virtual void CopyData(const CPDF_PageObject* pSrcObjet);
+ void CopyData(const CPDF_PageObject* pSrcObject) override;
};
+
class CPDF_ImageObject : public CPDF_PageObject
{
public:
-
CPDF_ImageObject();
virtual ~CPDF_ImageObject();
- virtual void Transform(const CFX_AffineMatrix& matrix);
+ void Transform(const CFX_AffineMatrix& matrix) override;
CPDF_Image* m_pImage;
CFX_AffineMatrix m_Matrix;
void CalcBoundingBox();
+
private:
- virtual void CopyData(const CPDF_PageObject* pSrcObjet);
+ void CopyData(const CPDF_PageObject* pSrcObject) override;
};
+
class CPDF_ShadingObject : public CPDF_PageObject
{
public:
-
CPDF_ShadingObject();
virtual ~CPDF_ShadingObject();
@@ -669,16 +670,17 @@ public:
CFX_AffineMatrix m_Matrix;
CPDF_Page* m_pPage;
- virtual void Transform(const CFX_AffineMatrix& matrix);
+ void Transform(const CFX_AffineMatrix& matrix) override;
void CalcBoundingBox();
+
protected:
- virtual void CopyData(const CPDF_PageObject* pSrcObjet);
+ void CopyData(const CPDF_PageObject* pSrcObject) override;
};
+
class CPDF_FormObject : public CPDF_PageObject
{
public:
-
CPDF_FormObject()
{
m_Type = PDFPAGE_FORM;
@@ -686,16 +688,18 @@ public:
}
virtual ~CPDF_FormObject();
- virtual void Transform(const CFX_AffineMatrix& matrix);
+ void Transform(const CFX_AffineMatrix& matrix) override;
CPDF_Form* m_pForm;
CFX_AffineMatrix m_FormMatrix;
void CalcBoundingBox();
+
protected:
- virtual void CopyData(const CPDF_PageObject* pSrcObjet);
+ void CopyData(const CPDF_PageObject* pSrcObject) override;
};
+
class CPDF_InlineImages : public CPDF_PageObject
{
public:
@@ -712,8 +716,7 @@ public:
void AddMatrix(CFX_AffineMatrix& matrix);
protected:
- virtual void Transform(const CFX_AffineMatrix& matrix) {}
- virtual void CopyData(const CPDF_PageObject* pSrcObjet) {}
+ void Transform(const CFX_AffineMatrix& matrix) override {}
};
#endif // CORE_INCLUDE_FPDFAPI_FPDF_PAGEOBJ_H_