Ensure that data is not overwritten in the audio input shared memory ring buffer that sits on the b…

content/browser/renderer_host/media/audio_input_sync_writer.h

 // 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.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_MEDIA_AUDIO_INPUT_SYNC_WRITER_H_
 #define CONTENT_BROWSER_RENDERER_HOST_MEDIA_AUDIO_INPUT_SYNC_WRITER_H_
 
 #include "base/memory/scoped_vector.h"
 #include "base/process/process.h"
 #include "base/sync_socket.h"
@@ skipping 52 matching lines @@
 
   // The time of the last Write call.
   base::Time last_write_time_;
 
   // Size in bytes of each audio bus.
   const int audio_bus_memory_size_;
 
   // Increasing ID used for audio buffers correct sequence at read side.
   uint32_t next_buffer_id_;
 
+  // Keeps track of number of expected buffer reads on the renderer side.
+  int expected_buffer_reads_;
+
+  // Counts the total number of buffer read verifictions and the number of
+  // timeouts when verifying.

[tommi (sloooow) - chröme, 2015/08/28 13:43:06]

is it possible to point to some design?  As is, it's hard to understand what
this actually means.

It would be good actually to document the design in a comment in this header.

+  size_t read_verification_count_;
+  size_t read_verification_timeout_count_;
+
   // Vector of audio buses allocated during construction and deleted in the
   // destructor.
   ScopedVector<media::AudioBus> audio_buses_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(AudioInputSyncWriter);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_MEDIA_AUDIO_INPUT_SYNC_WRITER_H_