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

Unified Diff: content/renderer/media/webmediaplayer_impl.cc

Issue 195973002: Change DCHECK_IS_ON() to DCHECK_IS_ON (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Keep a comment Created 6 years, 9 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
« no previous file with comments | « content/common/gpu/media/gpu_video_decode_accelerator.cc ('k') | gpu/command_buffer/service/test_helper.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/webmediaplayer_impl.cc
diff --git a/content/renderer/media/webmediaplayer_impl.cc b/content/renderer/media/webmediaplayer_impl.cc
index b7094f6c74259da75fc4b8451584d229647a75dc..2fa00322b6ac60bfbff281b48039f98d47878dc4 100644
--- a/content/renderer/media/webmediaplayer_impl.cc
+++ b/content/renderer/media/webmediaplayer_impl.cc
@@ -643,11 +643,11 @@ bool WebMediaPlayerImpl::copyVideoTextureToPlatformTexture(
// avoid it in user builds. As a result assume (below) that |texture| is
// bound when this method is called, and only verify this fact when
// DCHECK_IS_ON.
- if (DCHECK_IS_ON()) {
- GLint bound_texture = 0;
- web_graphics_context->getIntegerv(GL_TEXTURE_BINDING_2D, &bound_texture);
- DCHECK_EQ(static_cast<GLuint>(bound_texture), texture);
- }
+#if DCHECK_IS_ON
+ GLint bound_texture = 0;
+ web_graphics_context->getIntegerv(GL_TEXTURE_BINDING_2D, &bound_texture);
+ DCHECK_EQ(static_cast<GLuint>(bound_texture), texture);
+#endif
uint32 source_texture = web_graphics_context->createTexture();
« no previous file with comments | « content/common/gpu/media/gpu_video_decode_accelerator.cc ('k') | gpu/command_buffer/service/test_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698