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

Unified Diff: content/browser/renderer_host/media/audio_sync_reader.cc

Issue 2822123002: Avoid log spam in AudioSyncReader. (Closed)
Patch Set: Created 3 years, 8 months 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: content/browser/renderer_host/media/audio_sync_reader.cc
diff --git a/content/browser/renderer_host/media/audio_sync_reader.cc b/content/browser/renderer_host/media/audio_sync_reader.cc
index be6afd9c52278e59d6977244823a22764c9ac066..65eea8c9b60301850297891360381420ba457873 100644
--- a/content/browser/renderer_host/media/audio_sync_reader.cc
+++ b/content/browser/renderer_host/media/audio_sync_reader.cc
@@ -53,6 +53,7 @@ AudioSyncReader::AudioSyncReader(
: shared_memory_(std::move(shared_memory)),
mute_audio_(base::CommandLine::ForCurrentProcess()->HasSwitch(
switches::kMuteAudio)),
+ had_socket_error_(false),
socket_(std::move(socket)),
foreign_socket_(std::move(foreign_socket)),
packet_size_(shared_memory_->requested_size()),
@@ -175,9 +176,10 @@ void AudioSyncReader::RequestMoreData(base::TimeDelta delay,
}
size_t sent_bytes = socket_->Send(&control_signal, sizeof(control_signal));
- if (sent_bytes != sizeof(control_signal)) {
+ if (sent_bytes != sizeof(control_signal) && !had_socket_error_) {
+ had_socket_error_ = true;
tommi (sloooow) - chröme 2017/04/18 09:39:28 same here
Max Morin 2017/04/19 11:51:01 Done.
const std::string error_message = "ASR: No room in socket buffer.";
- LOG(WARNING) << error_message;
+ PLOG(WARNING) << error_message;
MediaStreamManager::SendMessageToNativeLog(error_message);
TRACE_EVENT_INSTANT0("audio",
"AudioSyncReader: No room in socket buffer",

Powered by Google App Engine
This is Rietveld 408576698