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_unittest.cc

Index: content/browser/renderer_host/media/audio_input_sync_writer_unittest.cc
diff --git a/content/browser/renderer_host/media/audio_input_sync_writer_unittest.cc b/content/browser/renderer_host/media/audio_input_sync_writer_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..f72e81922a7fd94f93898b9ce80da83a41f3f974
--- /dev/null
+++ b/content/browser/renderer_host/media/audio_input_sync_writer_unittest.cc
@@ -0,0 +1,172 @@
+// Copyright 2015 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 "base/memory/scoped_ptr.h"
+#include "base/sync_socket.h"
+#include "base/time/time.h"
+#include "content/browser/renderer_host/media/audio_input_sync_writer.h"
+#include "content/public/test/test_browser_thread_bundle.h"
+#include "media/audio/audio_parameters.h"
+#include "media/base/audio_bus.h"
+#include "media/base/channel_layout.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using ::testing::_;
+using base::TimeDelta;
+using media::AudioBus;
+using media::AudioParameters;
+
+namespace content {
+
+// Mocked out sockets used for Send/ReceiveWithTimeout. Counts the number of
+// outstanding reads, i.e. the diff between send and receive calls.
+class MockCancelableSyncSocket : public base::CancelableSyncSocket {
+ public:
+  MockCancelableSyncSocket(int max_pending_reads)
+      : in_failure_mode_(false),
+        pending_reads_(0),
+        max_pending_reads_(max_pending_reads) {}
+
+  size_t Send(const void* buffer, size_t length) override {
+    ++pending_reads_;
+    EXPECT_LE(pending_reads_, max_pending_reads_);
+    return length;
+  }
+
+  size_t ReceiveWithTimeout(void* buffer, size_t length,
+                            TimeDelta timeout) override {
+    if (in_failure_mode_)
+      return 0;
+
+    --pending_reads_;
+    EXPECT_GE(pending_reads_, 0);
+    return length;
+  }
+
+  // When |in_failure_mode_| == true, the socket fails to receive.
+  void SetFailureMode(bool in_failure_mode) {
+    in_failure_mode_ = in_failure_mode;
+  }
+
+  int get_pending_reads() { return pending_reads_; }
+
+ private:
+  bool in_failure_mode_;
+  int pending_reads_;
+  int max_pending_reads_;
+
+  DISALLOW_COPY_AND_ASSIGN(MockCancelableSyncSocket);
+};
+
+class AudioInputSyncWriterUnderTest : public AudioInputSyncWriter {
+ public:
+  AudioInputSyncWriterUnderTest(void* shared_memory,
+                                size_t shared_memory_size,
+                                int shared_memory_segment_count,
+                                const media::AudioParameters& params,
+                                base::CancelableSyncSocket* socket)
+      : AudioInputSyncWriter(shared_memory, shared_memory_size,
+                             shared_memory_segment_count, params) {
+    socket_.reset(socket);
+  }
+
+  ~AudioInputSyncWriterUnderTest() override {}
+
+  MOCK_METHOD1(AddToNativeLog, void(const std::string& message));
+};
+
+class AudioInputSyncWriterTest : public testing::Test {
+ public:
+  AudioInputSyncWriterTest()
+      : socket_(nullptr) {
+    const media::ChannelLayout layout =
+        media::ChannelLayout::CHANNEL_LAYOUT_MONO;
+    const int channels = media::ChannelLayoutToChannelCount(layout);
+    const int sampling_frequency_hz = 16000;
+    const int frames = sampling_frequency_hz / 100;  // 10 ms
+    const int bits_per_sample = 16;
+
+    AudioParameters audio_params(
+          AudioParameters::AUDIO_FAKE, layout, channels,
+          sampling_frequency_hz, bits_per_sample, frames);
+
+    const uint32 segment_size =
+        sizeof(media::AudioInputBufferParameters) +
+        AudioBus::CalculateMemorySize(audio_params);
+    size_t data_size = kSegments * segment_size;
+    data_.reset(new uint8[data_size]);
+
+    socket_ = new MockCancelableSyncSocket(kSegments);
+    writer_.reset(new AudioInputSyncWriterUnderTest(
+        data_.get(), data_size, kSegments, audio_params, socket_));
+    EXPECT_EQ(0, socket_->get_pending_reads());
+    audio_bus_ = AudioBus::Create(audio_params);
+  }
+
+  ~AudioInputSyncWriterTest() override {
+  }
+
+ protected:
+  const int kSegments = 10;
+
+  scoped_ptr<AudioInputSyncWriterUnderTest> writer_;
+  MockCancelableSyncSocket* socket_;
+  scoped_ptr<AudioBus> audio_bus_;
+  scoped_ptr<uint8[]> data_;
+
+ private:
+  TestBrowserThreadBundle thread_bundle_;
+
+  DISALLOW_COPY_AND_ASSIGN(AudioInputSyncWriterTest);
+};
+
+TEST_F(AudioInputSyncWriterTest, SingleWrite) {
+  EXPECT_CALL(*writer_.get(), AddToNativeLog(_));
+  writer_->Write(audio_bus_.get(), 0, false, 0);
+  EXPECT_EQ(1, socket_->get_pending_reads());
+}
+
+TEST_F(AudioInputSyncWriterTest, FillRingBufferOnce) {
+  // Write kSegments times. Pending reads should increase for each write.
+  EXPECT_CALL(*writer_.get(), AddToNativeLog(_));
+  for (int i = 1; i <= kSegments; ++i) {
+    writer_->Write(audio_bus_.get(), 0, false, 0);
+    EXPECT_EQ(i, socket_->get_pending_reads());
+  }
+}
+
+TEST_F(AudioInputSyncWriterTest, FillRingBufferOnceAndSingleWrite) {
+  EXPECT_CALL(*writer_.get(), AddToNativeLog(_));
+  for (int i = 1; i <= kSegments; ++i) {
+    writer_->Write(audio_bus_.get(), 0, false, 0);
+  }
+  EXPECT_EQ(kSegments, socket_->get_pending_reads());
+
+  // Next write should render in a receive and a send call on the socket,
+  // pending reads should remain constant after.
+  writer_->Write(audio_bus_.get(), 0, false, 0);
+  EXPECT_EQ(kSegments, socket_->get_pending_reads());
+}
+
+TEST_F(AudioInputSyncWriterTest, FillRingBufferOnceAndSingleWriteWithFailure) {
+  EXPECT_CALL(*writer_.get(), AddToNativeLog(_)).Times(2);
+  for (int i = 1; i <= kSegments; ++i) {
+    writer_->Write(audio_bus_.get(), 0, false, 0);
+  }
+  EXPECT_EQ(kSegments, socket_->get_pending_reads());
+
+  // Next write should render in a socket receive that fails, a log message and
+  // no socket send call on the socket, pending reads should remain constant
+  // after.
+  socket_->SetFailureMode(true);
+  writer_->Write(audio_bus_.get(), 0, false, 0);
+  EXPECT_EQ(kSegments, socket_->get_pending_reads());
+
+  socket_->SetFailureMode(false);
+  writer_->Write(audio_bus_.get(), 0, false, 0);
+  EXPECT_EQ(kSegments, socket_->get_pending_reads());
+}
+
+}  // namespace content