Silence detection test for the Windows audio capturer.

remoting/host/audio_capturer_win_unittest.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 "remoting/host/audio_capturer_win.h"
+#include "testing/gtest/include/gtest/gtest.h"

[Sergey Ulanov, 2012/08/21 20:52:15]

move this include after basictypes.h

[kxing, 2012/08/21 21:24:27]

Done.

+
+#include "base/basictypes.h"
+
+namespace remoting {
+
+class AudioCapturerWinTest : public testing::Test {

[Sergey Ulanov, 2012/08/21 20:52:15]

Don't really need this class. It's necessary only when you share code between
multiple tests. Remove it and use TEST() instead of TEST_F()

[kxing, 2012/08/21 21:24:27]

Done.

+};
+
+TEST_F(AudioCapturerWinTest, SilenceTest) {
+  int16 samples1[] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0};

[Sergey Ulanov, 2012/08/21 20:52:15]

Maybe add a test for an empty buffer?

[kxing, 2012/08/21 21:24:27]

Done.

+  int16 samples2[] = {65, 73, 83, 89, 92, -1, 5, 9, 123, 0};
+  int16 samples3[] = {0, 0, 0, 0, 1, 0, 0, -1, 0, 0};
+
+  ASSERT_TRUE(AudioCapturerWin::IsPacketOfSilence(samples1, 10));

[Sergey Ulanov, 2012/08/21 20:52:15]

here and below: s/10/arraysize(samples1)/

[kxing, 2012/08/21 21:24:27]

Done.

+  ASSERT_FALSE(AudioCapturerWin::IsPacketOfSilence(samples2, 10));
+  ASSERT_TRUE(AudioCapturerWin::IsPacketOfSilence(samples3, 10));
+}
+
+}  // namespace remoting