summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authortsepez <tsepez@chromium.org>2016-09-06 13:02:55 -0700
committerCommit bot <commit-bot@chromium.org>2016-09-06 13:02:55 -0700
commit7c9f0a11e3dd890aa2ae13d5bb5460951d2111d8 (patch)
treef3e393a0878d744744c69b445882632a4070d977
parentf692e8e33eb2e6811196170238471532edae6858 (diff)
downloadpdfium-7c9f0a11e3dd890aa2ae13d5bb5460951d2111d8.tar.xz
Move CPDF_GraphState to CFX_GraphState
The FX layer is spying on its private data, so make the job of ref-counting it live in FX as well, instead of up a layer at pdfapi. Pre-cursor to making this private and passing the graph state itself everywhere. Review-Url: https://codereview.chromium.org/2314223002
-rw-r--r--BUILD.gn4
-rw-r--r--core/fpdfapi/fpdf_page/cpdf_graphicstates.h4
-rw-r--r--core/fpdfapi/fpdf_render/fpdf_render.cpp2
-rw-r--r--core/fxge/ge/cfx_graphstate.cpp (renamed from core/fpdfapi/fpdf_page/cpdf_graphstate.cpp)32
-rw-r--r--core/fxge/include/cfx_graphstate.h (renamed from core/fpdfapi/fpdf_page/cpdf_graphstate.h)14
5 files changed, 28 insertions, 28 deletions
diff --git a/BUILD.gn b/BUILD.gn
index 0553f8b1a8..1223ef63b9 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -422,8 +422,6 @@ static_library("fpdfapi") {
"core/fpdfapi/fpdf_page/cpdf_generalstate.cpp",
"core/fpdfapi/fpdf_page/cpdf_graphicstates.cpp",
"core/fpdfapi/fpdf_page/cpdf_graphicstates.h",
- "core/fpdfapi/fpdf_page/cpdf_graphstate.cpp",
- "core/fpdfapi/fpdf_page/cpdf_graphstate.h",
"core/fpdfapi/fpdf_page/cpdf_image.cpp",
"core/fpdfapi/fpdf_page/cpdf_imageobject.cpp",
"core/fpdfapi/fpdf_page/cpdf_meshstream.cpp",
@@ -766,6 +764,7 @@ static_library("fxge") {
"core/fxge/ge/cfx_fontmapper.cpp",
"core/fxge/ge/cfx_fontmgr.cpp",
"core/fxge/ge/cfx_gemodule.cpp",
+ "core/fxge/ge/cfx_graphstate.cpp",
"core/fxge/ge/cfx_graphstatedata.cpp",
"core/fxge/ge/cfx_pathdata.cpp",
"core/fxge/ge/cfx_renderdevice.cpp",
@@ -785,6 +784,7 @@ static_library("fxge") {
"core/fxge/include/cfx_fontmgr.h",
"core/fxge/include/cfx_fxgedevice.h",
"core/fxge/include/cfx_gemodule.h",
+ "core/fxge/include/cfx_graphstate.h",
"core/fxge/include/cfx_graphstatedata.h",
"core/fxge/include/cfx_pathdata.h",
"core/fxge/include/cfx_renderdevice.h",
diff --git a/core/fpdfapi/fpdf_page/cpdf_graphicstates.h b/core/fpdfapi/fpdf_page/cpdf_graphicstates.h
index a408bc794b..fb404ffb72 100644
--- a/core/fpdfapi/fpdf_page/cpdf_graphicstates.h
+++ b/core/fpdfapi/fpdf_page/cpdf_graphicstates.h
@@ -8,10 +8,10 @@
#define CORE_FPDFAPI_FPDF_PAGE_CPDF_GRAPHICSTATES_H_
#include "core/fpdfapi/fpdf_page/cpdf_colorstate.h"
-#include "core/fpdfapi/fpdf_page/cpdf_graphstate.h"
#include "core/fpdfapi/fpdf_page/cpdf_textstate.h"
#include "core/fpdfapi/fpdf_page/include/cpdf_clippath.h"
#include "core/fpdfapi/fpdf_page/include/cpdf_generalstate.h"
+#include "core/fxge/include/cfx_graphstate.h"
class CPDF_GraphicStates {
public:
@@ -22,7 +22,7 @@ class CPDF_GraphicStates {
void DefaultStates();
CPDF_ClipPath m_ClipPath;
- CPDF_GraphState m_GraphState;
+ CFX_GraphState m_GraphState;
CPDF_ColorState m_ColorState;
CPDF_TextState m_TextState;
CPDF_GeneralState m_GeneralState;
diff --git a/core/fpdfapi/fpdf_render/fpdf_render.cpp b/core/fpdfapi/fpdf_render/fpdf_render.cpp
index d7c6e2f604..89667f3a23 100644
--- a/core/fpdfapi/fpdf_render/fpdf_render.cpp
+++ b/core/fpdfapi/fpdf_render/fpdf_render.cpp
@@ -493,7 +493,7 @@ FX_BOOL CPDF_RenderStatus::ProcessPath(CPDF_PathObject* pPathObj,
if (m_pType3Char)
FillType |= FX_FILL_TEXT_MODE;
- CPDF_GraphState graphState = pPathObj->m_GraphState;
+ CFX_GraphState graphState = pPathObj->m_GraphState;
if (m_Options.m_Flags & RENDER_THINLINE)
graphState.SetLineWidth(0);
return m_pDevice->DrawPathWithBlend(
diff --git a/core/fpdfapi/fpdf_page/cpdf_graphstate.cpp b/core/fxge/ge/cfx_graphstate.cpp
index 5a8dbe841a..9a6e73810f 100644
--- a/core/fpdfapi/fpdf_page/cpdf_graphstate.cpp
+++ b/core/fxge/ge/cfx_graphstate.cpp
@@ -4,24 +4,24 @@
// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
-#include "core/fpdfapi/fpdf_page/cpdf_graphstate.h"
+#include "core/fxge/include/cfx_graphstate.h"
#include "core/fpdfapi/fpdf_parser/include/cpdf_array.h"
-CPDF_GraphState::CPDF_GraphState() {}
+CFX_GraphState::CFX_GraphState() {}
-CPDF_GraphState::CPDF_GraphState(const CPDF_GraphState& that)
+CFX_GraphState::CFX_GraphState(const CFX_GraphState& that)
: m_Ref(that.m_Ref) {}
-CPDF_GraphState::~CPDF_GraphState() {}
+CFX_GraphState::~CFX_GraphState() {}
-void CPDF_GraphState::Emplace() {
+void CFX_GraphState::Emplace() {
m_Ref.Emplace();
}
-void CPDF_GraphState::SetLineDash(CPDF_Array* pArray,
- FX_FLOAT phase,
- FX_FLOAT scale) {
+void CFX_GraphState::SetLineDash(CPDF_Array* pArray,
+ FX_FLOAT phase,
+ FX_FLOAT scale) {
CFX_GraphStateData* pData = m_Ref.GetPrivateCopy();
pData->m_DashPhase = phase * scale;
pData->SetDashCount(static_cast<int>(pArray->GetCount()));
@@ -29,33 +29,33 @@ void CPDF_GraphState::SetLineDash(CPDF_Array* pArray,
pData->m_DashArray[i] = pArray->GetNumberAt(i) * scale;
}
-FX_FLOAT CPDF_GraphState::GetLineWidth() const {
+FX_FLOAT CFX_GraphState::GetLineWidth() const {
return m_Ref.GetObject()->m_LineWidth;
}
-void CPDF_GraphState::SetLineWidth(FX_FLOAT width) {
+void CFX_GraphState::SetLineWidth(FX_FLOAT width) {
m_Ref.GetPrivateCopy()->m_LineWidth = width;
}
-CFX_GraphStateData::LineCap CPDF_GraphState::GetLineCap() const {
+CFX_GraphStateData::LineCap CFX_GraphState::GetLineCap() const {
return m_Ref.GetObject()->m_LineCap;
}
-void CPDF_GraphState::SetLineCap(CFX_GraphStateData::LineCap cap) {
+void CFX_GraphState::SetLineCap(CFX_GraphStateData::LineCap cap) {
m_Ref.GetPrivateCopy()->m_LineCap = cap;
}
-CFX_GraphStateData::LineJoin CPDF_GraphState::GetLineJoin() const {
+CFX_GraphStateData::LineJoin CFX_GraphState::GetLineJoin() const {
return m_Ref.GetObject()->m_LineJoin;
}
-void CPDF_GraphState::SetLineJoin(CFX_GraphStateData::LineJoin join) {
+void CFX_GraphState::SetLineJoin(CFX_GraphStateData::LineJoin join) {
m_Ref.GetPrivateCopy()->m_LineJoin = join;
}
-FX_FLOAT CPDF_GraphState::GetMiterLimit() const {
+FX_FLOAT CFX_GraphState::GetMiterLimit() const {
return m_Ref.GetObject()->m_MiterLimit;
}
-void CPDF_GraphState::SetMiterLimit(FX_FLOAT limit) {
+void CFX_GraphState::SetMiterLimit(FX_FLOAT limit) {
m_Ref.GetPrivateCopy()->m_MiterLimit = limit;
}
diff --git a/core/fpdfapi/fpdf_page/cpdf_graphstate.h b/core/fxge/include/cfx_graphstate.h
index c3e3880f3c..9470303ac1 100644
--- a/core/fpdfapi/fpdf_page/cpdf_graphstate.h
+++ b/core/fxge/include/cfx_graphstate.h
@@ -4,19 +4,19 @@
// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
-#ifndef CORE_FPDFAPI_FPDF_PAGE_CPDF_GRAPHSTATE_H_
-#define CORE_FPDFAPI_FPDF_PAGE_CPDF_GRAPHSTATE_H_
+#ifndef CORE_FXGE_INCLUDE_CFX_GRAPHSTATE_H_
+#define CORE_FXGE_INCLUDE_CFX_GRAPHSTATE_H_
#include "core/fxcrt/include/cfx_count_ref.h"
#include "core/fxge/include/cfx_graphstatedata.h"
class CPDF_Array;
-class CPDF_GraphState {
+class CFX_GraphState {
public:
- CPDF_GraphState();
- CPDF_GraphState(const CPDF_GraphState& that);
- ~CPDF_GraphState();
+ CFX_GraphState();
+ CFX_GraphState(const CFX_GraphState& that);
+ ~CFX_GraphState();
void Emplace();
@@ -41,4 +41,4 @@ class CPDF_GraphState {
CFX_CountRef<CFX_GraphStateData> m_Ref;
};
-#endif // CORE_FPDFAPI_FPDF_PAGE_CPDF_GRAPHSTATE_H_
+#endif // CORE_FXGE_INCLUDE_CFX_GRAPHSTATE_H_