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/video_layer_impl.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/video_layer_impl.cc')
-rw-r--r-- | cc/video_layer_impl.cc | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/cc/video_layer_impl.cc b/cc/video_layer_impl.cc index cf5c048..dee1360 100644 --- a/cc/video_layer_impl.cc +++ b/cc/video_layer_impl.cc @@ -38,24 +38,24 @@ CCVideoLayerImpl::CCVideoLayerImpl(int id, WebKit::WebVideoFrameProvider* provid // thread is blocked. That makes this a thread-safe call to set the video // frame provider client that does not require a lock. The same is true of // the call in the destructor. - DCHECK(CCProxy::isMainThreadBlocked()); + ASSERT(CCProxy::isMainThreadBlocked()); m_provider->setVideoFrameProviderClient(this); } CCVideoLayerImpl::~CCVideoLayerImpl() { // See comment in constructor for why this doesn't need a lock. - DCHECK(CCProxy::isMainThreadBlocked()); + ASSERT(CCProxy::isMainThreadBlocked()); if (m_provider) { m_provider->setVideoFrameProviderClient(0); m_provider = 0; } freePlaneData(layerTreeHostImpl()->resourceProvider()); -#if CC_DCHECK_ENABLED() +#if !ASSERT_DISABLED for (unsigned i = 0; i < WebKit::WebVideoFrame::maxPlanes; ++i) - DCHECK(!m_framePlanes[i].resourceId); - DCHECK(!m_externalTextureResource); + ASSERT(!m_framePlanes[i].resourceId); + ASSERT(!m_externalTextureResource); #endif } @@ -64,7 +64,7 @@ void CCVideoLayerImpl::stopUsingProvider() // Block the provider from shutting down until this client is done // using the frame. base::AutoLock locker(m_providerLock); - DCHECK(!m_frame); + ASSERT(!m_frame); m_provider = 0; } @@ -88,7 +88,7 @@ static GC3Denum convertVFCFormatToGC3DFormat(const WebKit::WebVideoFrame& frame) void CCVideoLayerImpl::willDraw(CCResourceProvider* resourceProvider) { - DCHECK(CCProxy::isImplThread()); + ASSERT(CCProxy::isImplThread()); CCLayerImpl::willDraw(resourceProvider); // Explicitly acquire and release the provider mutex so it can be held from @@ -109,8 +109,8 @@ void CCVideoLayerImpl::willDraw(CCResourceProvider* resourceProvider) void CCVideoLayerImpl::willDrawInternal(CCResourceProvider* resourceProvider) { - DCHECK(CCProxy::isImplThread()); - DCHECK(!m_externalTextureResource); + ASSERT(CCProxy::isImplThread()); + ASSERT(!m_externalTextureResource); if (!m_provider) { m_frame = 0; @@ -154,7 +154,7 @@ void CCVideoLayerImpl::willDrawInternal(CCResourceProvider* resourceProvider) void CCVideoLayerImpl::appendQuads(CCQuadSink& quadSink, CCAppendQuadsData& appendQuadsData) { - DCHECK(CCProxy::isImplThread()); + ASSERT(CCProxy::isImplThread()); if (!m_frame) return; @@ -217,18 +217,18 @@ void CCVideoLayerImpl::appendQuads(CCQuadSink& quadSink, CCAppendQuadsData& appe void CCVideoLayerImpl::didDraw(CCResourceProvider* resourceProvider) { - DCHECK(CCProxy::isImplThread()); + ASSERT(CCProxy::isImplThread()); CCLayerImpl::didDraw(resourceProvider); if (!m_frame) return; if (m_format == GraphicsContext3D::TEXTURE_2D) { - DCHECK(m_externalTextureResource); + ASSERT(m_externalTextureResource); // FIXME: the following assert will not be true when sending resources to a // parent compositor. We will probably need to hold on to m_frame for // longer, and have several "current frames" in the pipeline. - DCHECK(!resourceProvider->inUseByConsumer(m_externalTextureResource)); + ASSERT(!resourceProvider->inUseByConsumer(m_externalTextureResource)); resourceProvider->deleteResource(m_externalTextureResource); m_externalTextureResource = 0; } |