diff options
author | danakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-12-04 07:17:40 +0000 |
---|---|---|
committer | danakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-12-04 07:17:40 +0000 |
commit | 2c5019c643e3b5b07aaf13f3967463a6d7dc4b00 (patch) | |
tree | 66cfb1b386a74dde95e7d62a497c8d378a7ee5de /webkit | |
parent | 6f4955bef249da68b6e704fe5d84c63d1456e950 (diff) | |
download | chromium_src-2c5019c643e3b5b07aaf13f3967463a6d7dc4b00.zip chromium_src-2c5019c643e3b5b07aaf13f3967463a6d7dc4b00.tar.gz chromium_src-2c5019c643e3b5b07aaf13f3967463a6d7dc4b00.tar.bz2 |
Remove DCHECK(context3d_) checks that check nothing.
We only allow creation of ContextProvider with a non-null context3d
and store it in the class inside the constructor. This was not always
the case, and we had DCHECKs to verify behaviour. Now, a DCHECK in
the constructor is enough (though also stricly not needed since the
Create() function checks for NULL). Remove the other DCHECKs as they
are just noise.
R=piman@chromium.org
BUG=
Review URL: https://codereview.chromium.org/102493003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238624 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit')
-rw-r--r-- | webkit/common/gpu/context_provider_in_process.cc | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/webkit/common/gpu/context_provider_in_process.cc b/webkit/common/gpu/context_provider_in_process.cc index dbb4ad8..f37bf90 100644 --- a/webkit/common/gpu/context_provider_in_process.cc +++ b/webkit/common/gpu/context_provider_in_process.cc @@ -145,7 +145,6 @@ ContextProviderInProcess::ContextCapabilities() { } blink::WebGraphicsContext3D* ContextProviderInProcess::Context3d() { - DCHECK(context3d_); DCHECK(lost_context_callback_proxy_); // Is bound to thread. DCHECK(context_thread_checker_.CalledOnValidThread()); @@ -153,7 +152,6 @@ blink::WebGraphicsContext3D* ContextProviderInProcess::Context3d() { } ::gpu::ContextSupport* ContextProviderInProcess::ContextSupport() { - DCHECK(context3d_); DCHECK(lost_context_callback_proxy_); // Is bound to thread. DCHECK(context_thread_checker_.CalledOnValidThread()); @@ -161,7 +159,6 @@ blink::WebGraphicsContext3D* ContextProviderInProcess::Context3d() { } class GrContext* ContextProviderInProcess::GrContext() { - DCHECK(context3d_); DCHECK(lost_context_callback_proxy_); // Is bound to thread. DCHECK(context_thread_checker_.CalledOnValidThread()); @@ -174,7 +171,6 @@ class GrContext* ContextProviderInProcess::GrContext() { } bool ContextProviderInProcess::IsContextLost() { - DCHECK(context3d_); DCHECK(lost_context_callback_proxy_); // Is bound to thread. DCHECK(context_thread_checker_.CalledOnValidThread()); @@ -182,7 +178,6 @@ bool ContextProviderInProcess::IsContextLost() { } void ContextProviderInProcess::VerifyContexts() { - DCHECK(context3d_); DCHECK(lost_context_callback_proxy_); // Is bound to thread. DCHECK(context_thread_checker_.CalledOnValidThread()); |