summaryrefslogtreecommitdiffstats
path: root/remoting/test/test_video_renderer.cc
diff options
context:
space:
mode:
authorjoedow <joedow@chromium.org>2015-07-06 11:44:18 -0700
committerCommit bot <commit-bot@chromium.org>2015-07-06 18:46:15 +0000
commit9cfae7cc7be9c99eb1b6a528d9393d9350901c63 (patch)
treef5f0783457fd39d447e20e5be54ca65fae75ce68 /remoting/test/test_video_renderer.cc
parenta52bc52af6c095e20cddb12afab4088ea99519d8 (diff)
downloadchromium_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.cc20
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()),