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

Unified Diff: remoting/codec/video_encoder_vpx.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/video_encoder_vpx.cc
diff --git a/remoting/codec/video_encoder_vpx.cc b/remoting/codec/video_encoder_vpx.cc
index 6ab4eb6e6c72c00887be3478d7dadb336433a2ea..352cc10e7ff897e0a57c2a7c94edd209f3f33527 100644
--- a/remoting/codec/video_encoder_vpx.cc
+++ b/remoting/codec/video_encoder_vpx.cc
@@ -403,10 +403,12 @@ void VideoEncoderVpx::Configure(const webrtc::DesktopSize& size) {
if (!codec_) {
codec_.reset(new vpx_codec_ctx_t);
ret = vpx_codec_enc_init(codec_.get(), interface, &config, 0);
- CHECK_EQ(VPX_CODEC_OK, ret) << "Failed to initialize codec";
+ // Failed to initialize codec
+ CHECK_EQ(VPX_CODEC_OK, ret);
} else {
ret = vpx_codec_enc_config_set(codec_.get(), &config);
- CHECK_EQ(VPX_CODEC_OK, ret) << "Failed to reconfigure codec";
+ // Failed to reconfigure codec
+ CHECK_EQ(VPX_CODEC_OK, ret);
}
// Apply further customizations to the codec now it's initialized.

Powered by Google App Engine
This is Rietveld 408576698