Avoid log spam in AudioSyncReader.

content/browser/renderer_host/media/audio_input_sync_writer.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/media/audio_input_sync_writer.h"
 
 #include <algorithm>
+#include <utility>
 
 #include "base/format_macros.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/stringprintf.h"
 #include "base/trace_event/trace_event.h"
 #include "build/build_config.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/public/browser/browser_thread.h"
 
 using media::AudioBus;
@@ skipping 22 matching lines @@
       shared_memory_segment_count_(shared_memory_segment_count),
       current_segment_id_(0),
       creation_time_(base::Time::Now()),
       audio_bus_memory_size_(AudioBus::CalculateMemorySize(params)),
       next_buffer_id_(0),
       next_read_buffer_index_(0),
       number_of_filled_segments_(0),
       write_count_(0),
       write_to_fifo_count_(0),
       write_error_count_(0),
+      had_socket_error_(false),
       trailing_write_to_fifo_count_(0),
       trailing_write_error_count_(0) {
   DCHECK_GT(shared_memory_segment_count, 0);
   DCHECK_EQ(shared_memory_size % shared_memory_segment_count, 0u);
   shared_memory_segment_size_ =
       shared_memory_size / shared_memory_segment_count;
   DVLOG(1) << "shared_memory_size: " << shared_memory_size;
   DVLOG(1) << "shared_memory_segment_count: " << shared_memory_segment_count;
   DVLOG(1) << "audio_bus_memory_size: " << audio_bus_memory_size_;
 
@@ skipping 261 matching lines @@
   buffer->params.volume = volume;
   buffer->params.size = audio_bus_memory_size_;
   buffer->params.key_pressed = key_pressed;
   buffer->params.hardware_delay_bytes = hardware_delay_bytes;
   buffer->params.id = next_buffer_id_;
 }
 
 bool AudioInputSyncWriter::SignalDataWrittenAndUpdateCounters() {
   if (socket_->Send(&current_segment_id_, sizeof(current_segment_id_)) !=
       sizeof(current_segment_id_)) {
-    const std::string error_message = "AISW: No room in socket buffer.";
-    LOG(WARNING) << error_message;
-    AddToNativeLog(error_message);
-    TRACE_EVENT_INSTANT0("audio",
-                         "AudioInputSyncWriter: No room in socket buffer",
-                         TRACE_EVENT_SCOPE_THREAD);
+    // Ensure we don't log consecutive errors as this can lead to a large
+    // amount of logs.
+    if (!had_socket_error_) {
+      had_socket_error_ = true;
+      const std::string error_message = "AISW: No room in socket buffer.";
+      PLOG(WARNING) << error_message;
+      AddToNativeLog(error_message);
+      TRACE_EVENT_INSTANT0("audio",
+                           "AudioInputSyncWriter: No room in socket buffer",
+                           TRACE_EVENT_SCOPE_THREAD);
+    }
     return false;
+  } else {
+    had_socket_error_ = false;
   }
 
   if (++current_segment_id_ >= shared_memory_segment_count_)
     current_segment_id_ = 0;
   ++number_of_filled_segments_;
   CHECK_LE(number_of_filled_segments_,
            static_cast<int>(shared_memory_segment_count_));
   ++next_buffer_id_;
 
   return true;
 }
 
 }  // namespace content