diff options
author | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-17 01:25:18 +0000 |
---|---|---|
committer | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-10-17 01:25:18 +0000 |
commit | e7f87cfba8e6f916dd88359ed9abf6ec6884d65d (patch) | |
tree | 289b3ec335cf58e6924f5a6fccd8e17ea1c6585a /cc/occlusion_tracker.cc | |
parent | bc3847a1ba39db7adec20aa9e0565f6e868e8480 (diff) | |
download | chromium_src-e7f87cfba8e6f916dd88359ed9abf6ec6884d65d.zip chromium_src-e7f87cfba8e6f916dd88359ed9abf6ec6884d65d.tar.gz chromium_src-e7f87cfba8e6f916dd88359ed9abf6ec6884d65d.tar.bz2 |
Revert "cc: Switch to Chromium DCHECKs LOGs"
This reverts commit 162296. (https://chromiumcodereview.appspot.com/11048044)
TBR=danakj@chromium.org
Review URL: https://codereview.chromium.org/11196014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162297 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/occlusion_tracker.cc')
-rw-r--r-- | cc/occlusion_tracker.cc | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/cc/occlusion_tracker.cc b/cc/occlusion_tracker.cc index 7d5640d..39d7efe 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) { - DCHECK(layer->visibleContentRect().contains(opaqueContents.bounds())); + ASSERT(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) { - DCHECK(!m_stack.isEmpty()); - DCHECK(layer->renderTarget() == m_stack.last().target); + ASSERT(!m_stack.isEmpty()); + ASSERT(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; - DCHECK(!m_stack.isEmpty()); + ASSERT(!m_stack.isEmpty()); if (m_stack.isEmpty()) return false; if (contentRect.isEmpty()) return true; - DCHECK(layer->renderTarget() == m_stack.last().target); + ASSERT(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 { - DCHECK(!m_stack.isEmpty()); + ASSERT(!m_stack.isEmpty()); if (m_stack.isEmpty()) return contentRect; if (contentRect.isEmpty()) return contentRect; - DCHECK(layer->renderTarget() == m_stack.last().target); + ASSERT(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 { - DCHECK(!m_stack.isEmpty()); + ASSERT(!m_stack.isEmpty()); // The layer is a contributing renderTarget so it should have a surface. - DCHECK(layer->renderSurface()); + ASSERT(layer->renderSurface()); // The layer is a contributing renderTarget so its target should be itself. - DCHECK(layer->renderTarget() == layer); + ASSERT(layer->renderTarget() == layer); // The layer should not be the root, else what is is contributing to? - DCHECK(layer->parent()); + ASSERT(layer->parent()); // This should be called while the layer is still considered the current target in the occlusion tracker. - DCHECK(layer == m_stack.last().target); + ASSERT(layer == m_stack.last().target); if (contentRect.isEmpty()) return contentRect; |