diff options
| author | Lei Zhang <thestig@chromium.org> | 2017-05-10 14:00:53 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-05-10 21:08:04 +0000 |
| commit | 0a0892626d24ce82e7026c32c71b1de036d4bbe1 (patch) | |
| tree | 470bca7e1c5b27c967d246706d21018fd4d2d1f2 /core/fpdfapi/render | |
| parent | 6f62ccd50ff85ba6a60ddf47993745739854cd8f (diff) | |
| download | pdfium-0a0892626d24ce82e7026c32c71b1de036d4bbe1.tar.xz | |
Replace operator bool with HasRef() in classes with a CFX_SharedCopyOnWrite member.
Change-Id: I51e30d298e87b9ae0d5aca83b2f1d6787efce70a
Reviewed-on: https://pdfium-review.googlesource.com/5290
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Nicolás Peña <npm@chromium.org>
Diffstat (limited to 'core/fpdfapi/render')
| -rw-r--r-- | core/fpdfapi/render/cpdf_imagerenderer.cpp | 2 | ||||
| -rw-r--r-- | core/fpdfapi/render/cpdf_renderstatus.cpp | 18 |
2 files changed, 10 insertions, 10 deletions
diff --git a/core/fpdfapi/render/cpdf_imagerenderer.cpp b/core/fpdfapi/render/cpdf_imagerenderer.cpp index 6562e154fc..dd1044816a 100644 --- a/core/fpdfapi/render/cpdf_imagerenderer.cpp +++ b/core/fpdfapi/render/cpdf_imagerenderer.cpp @@ -133,7 +133,7 @@ bool CPDF_ImageRenderer::StartRenderDIBSource() { if (m_bPatternColor) return DrawPatternImage(m_pObj2Device); - if (m_BitmapAlpha != 255 || !state || !state.GetFillOP() || + if (m_BitmapAlpha != 255 || !state.HasRef() || !state.GetFillOP() || state.GetOPMode() != 0 || state.GetBlendType() != FXDIB_BLEND_NORMAL || state.GetStrokeAlpha() != 1.0f || state.GetFillAlpha() != 1.0f) { return StartDIBSource(); diff --git a/core/fpdfapi/render/cpdf_renderstatus.cpp b/core/fpdfapi/render/cpdf_renderstatus.cpp index 0a288c4b00..44af9a2bac 100644 --- a/core/fpdfapi/render/cpdf_renderstatus.cpp +++ b/core/fpdfapi/render/cpdf_renderstatus.cpp @@ -1068,7 +1068,7 @@ void CPDF_RenderStatus::RenderSingleObject(CPDF_PageObject* pObj, return; } m_pCurObj = pObj; - if (m_Options.m_pOCContext && pObj->m_ContentMark) { + if (m_Options.m_pOCContext && pObj->m_ContentMark.HasRef()) { if (!m_Options.m_pOCContext->CheckObjectVisible(pObj)) { return; } @@ -1097,7 +1097,7 @@ bool CPDF_RenderStatus::ContinueSingleObject(CPDF_PageObject* pObj, } m_pCurObj = pObj; - if (m_Options.m_pOCContext && pObj->m_ContentMark && + if (m_Options.m_pOCContext && pObj->m_ContentMark.HasRef() && !m_Options.m_pOCContext->CheckObjectVisible(pObj)) { return false; } @@ -1315,10 +1315,10 @@ FX_ARGB CPDF_RenderStatus::GetFillArgb(CPDF_PageObject* pObj, if (m_pType3Char && !bType3 && (!m_pType3Char->m_bColored || (m_pType3Char->m_bColored && - (!*pColorState || pColorState->GetFillColor()->IsNull())))) { + (!pColorState->HasRef() || pColorState->GetFillColor()->IsNull())))) { return m_T3FillColor; } - if (!*pColorState || pColorState->GetFillColor()->IsNull()) + if (!pColorState->HasRef() || pColorState->GetFillColor()->IsNull()) pColorState = &m_InitialStates.m_ColorState; FX_COLORREF rgb = pColorState->GetFillRGB(); @@ -1343,10 +1343,10 @@ FX_ARGB CPDF_RenderStatus::GetStrokeArgb(CPDF_PageObject* pObj) const { if (m_pType3Char && (!m_pType3Char->m_bColored || (m_pType3Char->m_bColored && - (!*pColorState || pColorState->GetStrokeColor()->IsNull())))) { + (!pColorState->HasRef() || pColorState->GetStrokeColor()->IsNull())))) { return m_T3FillColor; } - if (!*pColorState || pColorState->GetStrokeColor()->IsNull()) + if (!pColorState->HasRef() || pColorState->GetStrokeColor()->IsNull()) pColorState = &m_InitialStates.m_ColorState; FX_COLORREF rgb = pColorState->GetStrokeRGB(); @@ -1368,8 +1368,8 @@ FX_ARGB CPDF_RenderStatus::GetStrokeArgb(CPDF_PageObject* pObj) const { void CPDF_RenderStatus::ProcessClipPath(CPDF_ClipPath ClipPath, const CFX_Matrix* pObj2Device) { - if (!ClipPath) { - if (m_LastClipPath) { + if (!ClipPath.HasRef()) { + if (m_LastClipPath.HasRef()) { m_pDevice->RestoreState(true); m_LastClipPath.SetNull(); } @@ -1477,7 +1477,7 @@ bool CPDF_RenderStatus::ProcessTransparency(CPDF_PageObject* pPageObj, } } bool bTextClip = - (pPageObj->m_ClipPath && pPageObj->m_ClipPath.GetTextCount() && + (pPageObj->m_ClipPath.HasRef() && pPageObj->m_ClipPath.GetTextCount() && m_pDevice->GetDeviceClass() == FXDC_DISPLAY && !(m_pDevice->GetDeviceCaps(FXDC_RENDER_CAPS) & FXRC_SOFT_CLIP)); if ((m_Options.m_Flags & RENDER_OVERPRINT) && pPageObj->IsImage() && |