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

Unified Diff: remoting/codec/scoped_vpx_codec.cc

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 years 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: remoting/codec/scoped_vpx_codec.cc
diff --git a/remoting/codec/scoped_vpx_codec.cc b/remoting/codec/scoped_vpx_codec.cc
index dda986e75d272aa0e165398b3135ada6fa442c58..c4591bf0e1f85d6598a8238a1a6bf3ba04a488a3 100644
--- a/remoting/codec/scoped_vpx_codec.cc
+++ b/remoting/codec/scoped_vpx_codec.cc
@@ -15,7 +15,8 @@ namespace remoting {
void VpxCodecDeleter::operator()(vpx_codec_ctx_t* codec) {
if (codec) {
vpx_codec_err_t ret = vpx_codec_destroy(codec);
- CHECK_EQ(ret, VPX_CODEC_OK) << "Failed to destroy codec";
+ // Failed to destroy codec
+ CHECK_EQ(ret, VPX_CODEC_OK);
delete codec;
}
}

Powered by Google App Engine
This is Rietveld 408576698