diff options
author | joedow <joedow@chromium.org> | 2015-07-06 11:44:18 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-06 18:46:15 +0000 |
commit | 9cfae7cc7be9c99eb1b6a528d9393d9350901c63 (patch) | |
tree | f5f0783457fd39d447e20e5be54ca65fae75ce68 /remoting/test/test_video_renderer.cc | |
parent | a52bc52af6c095e20cddb12afab4088ea99519d8 (diff) | |
download | chromium_src-9cfae7cc7be9c99eb1b6a528d9393d9350901c63.zip chromium_src-9cfae7cc7be9c99eb1b6a528d9393d9350901c63.tar.gz chromium_src-9cfae7cc7be9c99eb1b6a528d9393d9350901c63.tar.bz2 |
Updating App Remoting Test Driver logging behavior.
I am changing the log statements in the app remoting test driver to use
VLOG instead of DVLOG. The reason for this is that we build and run the
release version of the tool on the chromoting waterfall and there isn't
enough information in the normal output to debug failures when they
occur. The other option is to run the debug version of the tool on the
waterfall but I think this will be better in the log term (helps
debugging both versions of the tool).
BUG=
Review URL: https://codereview.chromium.org/1215863004
Cr-Commit-Position: refs/heads/master@{#337426}
Diffstat (limited to 'remoting/test/test_video_renderer.cc')
-rw-r--r-- | remoting/test/test_video_renderer.cc | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/remoting/test/test_video_renderer.cc b/remoting/test/test_video_renderer.cc index 908bb46..b1ae5fa 100644 --- a/remoting/test/test_video_renderer.cc +++ b/remoting/test/test_video_renderer.cc @@ -86,17 +86,17 @@ void TestVideoRenderer::Core::SetCodecForDecoding( switch (codec) { case protocol::ChannelConfig::CODEC_VP8: { - DVLOG(2) << "Test Video Renderer will use VP8 decoder"; + VLOG(1) << "Test Video Renderer will use VP8 decoder"; decoder_ = VideoDecoderVpx::CreateForVP8(); break; } case protocol::ChannelConfig::CODEC_VP9: { - DVLOG(2) << "Test Video Renderer will use VP9 decoder"; + VLOG(1) << "Test Video Renderer will use VP9 decoder"; decoder_ = VideoDecoderVpx::CreateForVP9(); break; } case protocol::ChannelConfig::CODEC_VERBATIM: { - DVLOG(2) << "Test Video Renderer will use VERBATIM decoder"; + VLOG(1) << "Test Video Renderer will use VERBATIM decoder"; decoder_.reset(new VideoDecoderVerbatim()); break; } @@ -119,7 +119,7 @@ void TestVideoRenderer::Core::ProcessVideoPacket( DCHECK(decoder_); DCHECK(packet); - DVLOG(2) << "TestVideoRenderer::Core::ProcessVideoPacket() Called"; + VLOG(2) << "TestVideoRenderer::Core::ProcessVideoPacket() Called"; // Screen size is attached on the first packet as well as when the // host screen is resized. @@ -184,7 +184,7 @@ TestVideoRenderer::~TestVideoRenderer() { void TestVideoRenderer::OnSessionConfig(const protocol::SessionConfig& config) { DCHECK(thread_checker_.CalledOnValidThread()); - DVLOG(2) << "TestVideoRenderer::OnSessionConfig() Called"; + VLOG(2) << "TestVideoRenderer::OnSessionConfig() Called"; protocol::ChannelConfig::Codec codec = config.video_config().codec; SetCodecForDecoding(codec); } @@ -192,14 +192,14 @@ void TestVideoRenderer::OnSessionConfig(const protocol::SessionConfig& config) { ChromotingStats* TestVideoRenderer::GetStats() { DCHECK(thread_checker_.CalledOnValidThread()); - DVLOG(2) << "TestVideoRenderer::GetStats() Called"; + VLOG(2) << "TestVideoRenderer::GetStats() Called"; return nullptr; } protocol::VideoStub* TestVideoRenderer::GetVideoStub() { DCHECK(thread_checker_.CalledOnValidThread()); - DVLOG(2) << "TestVideoRenderer::GetVideoStub() Called"; + VLOG(2) << "TestVideoRenderer::GetVideoStub() Called"; return this; } @@ -209,7 +209,7 @@ void TestVideoRenderer::ProcessVideoPacket(scoped_ptr<VideoPacket> video_packet, DCHECK(video_decode_task_runner_) << "Failed to start video decode thread"; if (video_packet->has_data() && video_packet->data().size() != 0) { - DVLOG(2) << "process video packet is called!"; + VLOG(2) << "process video packet is called!"; // Post video process task to the video decode thread. base::Closure process_video_task = base::Bind( @@ -219,7 +219,7 @@ void TestVideoRenderer::ProcessVideoPacket(scoped_ptr<VideoPacket> video_packet, } else { // Log at a high verbosity level as we receive empty packets frequently and // they can clutter up the debug output if the level is set too low. - DVLOG(3) << "Empty Video Packet received."; + VLOG(3) << "Empty Video Packet received."; done.Run(); } } @@ -228,7 +228,7 @@ void TestVideoRenderer::SetCodecForDecoding( const protocol::ChannelConfig::Codec codec) { DCHECK(thread_checker_.CalledOnValidThread()); - DVLOG(2) << "TestVideoRenderer::SetDecoder() Called"; + VLOG(2) << "TestVideoRenderer::SetDecoder() Called"; video_decode_task_runner_->PostTask( FROM_HERE, base::Bind(&Core::SetCodecForDecoding, base::Unretained(core_.get()), |