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

Unified Diff: cc/occlusion_tracker.cc

Issue 11048044: cc: Switch to Chromium DCHECKs and LOGs (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: with-presubmit Created 8 years, 2 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: cc/occlusion_tracker.cc
diff --git a/cc/occlusion_tracker.cc b/cc/occlusion_tracker.cc
index debc5114398357d1be8ebc422818c0aeeddb8b80..be49749eeee0d785ec4c53302282eb9d4af2673c 100644
--- a/cc/occlusion_tracker.cc
+++ b/cc/occlusion_tracker.cc
@@ -245,7 +245,7 @@ void CCOcclusionTrackerBase<LayerType, RenderSurfaceType>::leaveToRenderTarget(c
template<typename LayerType>
static inline void addOcclusionBehindLayer(Region& region, const LayerType* layer, const WebTransformationMatrix& transform, const Region& opaqueContents, const IntRect& clipRectInTarget, const IntSize& minimumTrackingSize, Vector<IntRect>* occludingScreenSpaceRects)
{
- ASSERT(layer->visibleContentRect().contains(opaqueContents.bounds()));
+ CC_DCHECK(layer->visibleContentRect().contains(opaqueContents.bounds()));
bool clipped;
FloatQuad visibleTransformedQuad = CCMathUtil::mapQuad(transform, FloatQuad(layer->visibleContentRect()), clipped);
@@ -269,8 +269,8 @@ static inline void addOcclusionBehindLayer(Region& region, const LayerType* laye
template<typename LayerType, typename RenderSurfaceType>
void CCOcclusionTrackerBase<LayerType, RenderSurfaceType>::markOccludedBehindLayer(const LayerType* layer)
{
- ASSERT(!m_stack.isEmpty());
- ASSERT(layer->renderTarget() == m_stack.last().target);
+ CC_DCHECK(!m_stack.isEmpty());
+ CC_DCHECK(layer->renderTarget() == m_stack.last().target);
if (m_stack.isEmpty())
return;
@@ -317,13 +317,13 @@ bool CCOcclusionTrackerBase<LayerType, RenderSurfaceType>::occluded(const LayerT
if (hasOcclusionFromOutsideTargetSurface)
*hasOcclusionFromOutsideTargetSurface = false;
- ASSERT(!m_stack.isEmpty());
+ CC_DCHECK(!m_stack.isEmpty());
if (m_stack.isEmpty())
return false;
if (contentRect.isEmpty())
return true;
- ASSERT(layer->renderTarget() == m_stack.last().target);
+ CC_DCHECK(layer->renderTarget() == m_stack.last().target);
if (layerTransformsToTargetKnown(layer) && testContentRectOccluded(contentRect, layer->drawTransform(), layerClipRectInTarget(layer), m_stack.last().occlusionInTarget))
return true;
@@ -362,13 +362,13 @@ static inline IntRect computeUnoccludedContentRect(const IntRect& contentRect, c
template<typename LayerType, typename RenderSurfaceType>
IntRect CCOcclusionTrackerBase<LayerType, RenderSurfaceType>::unoccludedContentRect(const LayerType* layer, const IntRect& contentRect, bool* hasOcclusionFromOutsideTargetSurface) const
{
- ASSERT(!m_stack.isEmpty());
+ CC_DCHECK(!m_stack.isEmpty());
if (m_stack.isEmpty())
return contentRect;
if (contentRect.isEmpty())
return contentRect;
- ASSERT(layer->renderTarget() == m_stack.last().target);
+ CC_DCHECK(layer->renderTarget() == m_stack.last().target);
// We want to return a rect that contains all the visible parts of |contentRect| in both screen space and in the target surface.
// So we find the visible parts of |contentRect| in each space, and take the intersection.
@@ -390,15 +390,15 @@ IntRect CCOcclusionTrackerBase<LayerType, RenderSurfaceType>::unoccludedContentR
template<typename LayerType, typename RenderSurfaceType>
IntRect CCOcclusionTrackerBase<LayerType, RenderSurfaceType>::unoccludedContributingSurfaceContentRect(const LayerType* layer, bool forReplica, const IntRect& contentRect, bool* hasOcclusionFromOutsideTargetSurface) const
{
- ASSERT(!m_stack.isEmpty());
+ CC_DCHECK(!m_stack.isEmpty());
// The layer is a contributing renderTarget so it should have a surface.
- ASSERT(layer->renderSurface());
+ CC_DCHECK(layer->renderSurface());
// The layer is a contributing renderTarget so its target should be itself.
- ASSERT(layer->renderTarget() == layer);
+ CC_DCHECK(layer->renderTarget() == layer);
// The layer should not be the root, else what is is contributing to?
- ASSERT(layer->parent());
+ CC_DCHECK(layer->parent());
// This should be called while the layer is still considered the current target in the occlusion tracker.
- ASSERT(layer == m_stack.last().target);
+ CC_DCHECK(layer == m_stack.last().target);
if (contentRect.isEmpty())
return contentRect;

Powered by Google App Engine
This is Rietveld 408576698