diff options
author | tsepez <tsepez@chromium.org> | 2016-09-06 13:02:55 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-09-06 13:02:55 -0700 |
commit | 7c9f0a11e3dd890aa2ae13d5bb5460951d2111d8 (patch) | |
tree | f3e393a0878d744744c69b445882632a4070d977 /core/fpdfapi/fpdf_page | |
parent | f692e8e33eb2e6811196170238471532edae6858 (diff) | |
download | pdfium-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
Diffstat (limited to 'core/fpdfapi/fpdf_page')
-rw-r--r-- | core/fpdfapi/fpdf_page/cpdf_graphicstates.h | 4 | ||||
-rw-r--r-- | core/fpdfapi/fpdf_page/cpdf_graphstate.cpp | 61 | ||||
-rw-r--r-- | core/fpdfapi/fpdf_page/cpdf_graphstate.h | 44 |
3 files changed, 2 insertions, 107 deletions
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_page/cpdf_graphstate.cpp b/core/fpdfapi/fpdf_page/cpdf_graphstate.cpp deleted file mode 100644 index 5a8dbe841a..0000000000 --- a/core/fpdfapi/fpdf_page/cpdf_graphstate.cpp +++ /dev/null @@ -1,61 +0,0 @@ -// Copyright 2016 PDFium Authors. All rights reserved. -// Use of this source code is governed by a BSD-style license that can be -// found in the LICENSE file. - -// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com - -#include "core/fpdfapi/fpdf_page/cpdf_graphstate.h" - -#include "core/fpdfapi/fpdf_parser/include/cpdf_array.h" - -CPDF_GraphState::CPDF_GraphState() {} - -CPDF_GraphState::CPDF_GraphState(const CPDF_GraphState& that) - : m_Ref(that.m_Ref) {} - -CPDF_GraphState::~CPDF_GraphState() {} - -void CPDF_GraphState::Emplace() { - m_Ref.Emplace(); -} - -void CPDF_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())); - for (size_t i = 0; i < pArray->GetCount(); i++) - pData->m_DashArray[i] = pArray->GetNumberAt(i) * scale; -} - -FX_FLOAT CPDF_GraphState::GetLineWidth() const { - return m_Ref.GetObject()->m_LineWidth; -} - -void CPDF_GraphState::SetLineWidth(FX_FLOAT width) { - m_Ref.GetPrivateCopy()->m_LineWidth = width; -} - -CFX_GraphStateData::LineCap CPDF_GraphState::GetLineCap() const { - return m_Ref.GetObject()->m_LineCap; -} -void CPDF_GraphState::SetLineCap(CFX_GraphStateData::LineCap cap) { - m_Ref.GetPrivateCopy()->m_LineCap = cap; -} - -CFX_GraphStateData::LineJoin CPDF_GraphState::GetLineJoin() const { - return m_Ref.GetObject()->m_LineJoin; -} - -void CPDF_GraphState::SetLineJoin(CFX_GraphStateData::LineJoin join) { - m_Ref.GetPrivateCopy()->m_LineJoin = join; -} - -FX_FLOAT CPDF_GraphState::GetMiterLimit() const { - return m_Ref.GetObject()->m_MiterLimit; -} - -void CPDF_GraphState::SetMiterLimit(FX_FLOAT limit) { - m_Ref.GetPrivateCopy()->m_MiterLimit = limit; -} diff --git a/core/fpdfapi/fpdf_page/cpdf_graphstate.h b/core/fpdfapi/fpdf_page/cpdf_graphstate.h deleted file mode 100644 index c3e3880f3c..0000000000 --- a/core/fpdfapi/fpdf_page/cpdf_graphstate.h +++ /dev/null @@ -1,44 +0,0 @@ -// Copyright 2016 PDFium Authors. All rights reserved. -// Use of this source code is governed by a BSD-style license that can be -// found in the LICENSE file. - -// 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_ - -#include "core/fxcrt/include/cfx_count_ref.h" -#include "core/fxge/include/cfx_graphstatedata.h" - -class CPDF_Array; - -class CPDF_GraphState { - public: - CPDF_GraphState(); - CPDF_GraphState(const CPDF_GraphState& that); - ~CPDF_GraphState(); - - void Emplace(); - - void SetLineDash(CPDF_Array* pArray, FX_FLOAT phase, FX_FLOAT scale); - - FX_FLOAT GetLineWidth() const; - void SetLineWidth(FX_FLOAT width); - - CFX_GraphStateData::LineCap GetLineCap() const; - void SetLineCap(CFX_GraphStateData::LineCap cap); - - CFX_GraphStateData::LineJoin GetLineJoin() const; - void SetLineJoin(CFX_GraphStateData::LineJoin join); - - FX_FLOAT GetMiterLimit() const; - void SetMiterLimit(FX_FLOAT limit); - - // FIXME(tsepez): remove when all GraphStateData usage gone. - const CFX_GraphStateData* GetObject() const { return m_Ref.GetObject(); } - - private: - CFX_CountRef<CFX_GraphStateData> m_Ref; -}; - -#endif // CORE_FPDFAPI_FPDF_PAGE_CPDF_GRAPHSTATE_H_ |