diff options
author | hguihot@google.com <hguihot@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-12-13 09:28:45 +0000 |
---|---|---|
committer | hguihot@google.com <hguihot@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-12-13 09:28:45 +0000 |
commit | 2a5f3f55076eb774be0aa57c8786f2c6e7612d0b (patch) | |
tree | 9db206780c0a4cd1987f9b5d5c050ebb60eadc65 /media/cast/audio_receiver/audio_receiver.cc | |
parent | 457d695943b205d852e9eb9c8b03d4d1bbb8caa7 (diff) | |
download | chromium_src-2a5f3f55076eb774be0aa57c8786f2c6e7612d0b.zip chromium_src-2a5f3f55076eb774be0aa57c8786f2c6e7612d0b.tar.gz chromium_src-2a5f3f55076eb774be0aa57c8786f2c6e7612d0b.tar.bz2 |
Change VLOG levels in media/cast.
Convention:
VLOG(1) for information that's unlikely to flood logs.
VLOG(2) for per-frame information.
VLOG(3) for per-packet information (likely to flood logs).
Certain logs are 'promoted'. For example, per-frame information can
still use VLOG(1) if it does not occur that often or should be
considered important enough to use VLOG(1).
Review URL: https://codereview.chromium.org/104403010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@240570 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/cast/audio_receiver/audio_receiver.cc')
-rw-r--r-- | media/cast/audio_receiver/audio_receiver.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/media/cast/audio_receiver/audio_receiver.cc b/media/cast/audio_receiver/audio_receiver.cc index d1f9e13..39c02c9 100644 --- a/media/cast/audio_receiver/audio_receiver.cc +++ b/media/cast/audio_receiver/audio_receiver.cc @@ -176,7 +176,7 @@ void AudioReceiver::IncomingParsedRtpPacket(const uint8* payload_data, } if (!decryptor_->Decrypt(base::StringPiece(reinterpret_cast<const char*>( payload_data), payload_size), &plaintext)) { - VLOG(0) << "Decryption error"; + LOG(ERROR) << "Decryption error"; return; } } @@ -287,7 +287,7 @@ void AudioReceiver::PlayoutTimeout() { // Since the application can post multiple AudioFrameEncodedCallback and // we only check the next frame to play out we might have multiple timeout // events firing after each other; however this should be a rare event. - VLOG(1) << "Failed to retrieved a complete frame at this point in time"; + VLOG(2) << "Failed to retrieved a complete frame at this point in time"; return; } @@ -315,7 +315,7 @@ void AudioReceiver::GetEncodedAudioFrame( if (!audio_buffer_->GetEncodedAudioFrame(encoded_frame.get(), &rtp_timestamp, &next_frame)) { // We have no audio frames. Wait for new packet(s). - VLOG(1) << "Wait for more audio packets in frame"; + VLOG(2) << "Wait for more audio packets in frame"; queued_encoded_callbacks_.push_back(callback); return; } @@ -351,7 +351,7 @@ bool AudioReceiver::PostEncodedAudioFrame( cast_environment_->PostDelayedTask(CastEnvironment::MAIN, FROM_HERE, base::Bind(&AudioReceiver::PlayoutTimeout, weak_factory_.GetWeakPtr()), time_until_release); - VLOG(1) << "Wait until time to playout:" + VLOG(2) << "Wait until time to playout:" << time_until_release.InMilliseconds(); return false; } @@ -430,7 +430,7 @@ bool AudioReceiver::DecryptAudioFrame( } std::string decrypted_audio_data; if (!decryptor_->Decrypt((*audio_frame)->data, &decrypted_audio_data)) { - VLOG(0) << "Decryption error"; + LOG(ERROR) << "Decryption error"; // Give up on this frame, release it from jitter buffer. audio_buffer_->ReleaseFrame((*audio_frame)->frame_id); return false; |