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

Unified Diff: remoting/codec/webrtc_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/webrtc_video_encoder_vpx.cc
diff --git a/remoting/codec/webrtc_video_encoder_vpx.cc b/remoting/codec/webrtc_video_encoder_vpx.cc
index dfa6050d79c5d504868ccd350cc587fd4b8c73ab..25da48d66a995f0c78a735fddd4ae5dc75c32f8b 100644
--- a/remoting/codec/webrtc_video_encoder_vpx.cc
+++ b/remoting/codec/webrtc_video_encoder_vpx.cc
@@ -420,10 +420,12 @@ void WebrtcVideoEncoderVpx::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