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

Unified Diff: media/audio/audio_input_controller_unittest.cc

Issue 6628020: Cleaning up src/media to be consistent with static versus anonymous namespaces. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src
Patch Set: fix namespaces Created 9 years, 10 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: media/audio/audio_input_controller_unittest.cc
diff --git a/media/audio/audio_input_controller_unittest.cc b/media/audio/audio_input_controller_unittest.cc
index b6efe96dac23df687cf21889dcbb9807d319ad67..9a658d4796759a60ba2aec5c74a3e49ce6924d83 100644
--- a/media/audio/audio_input_controller_unittest.cc
+++ b/media/audio/audio_input_controller_unittest.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
@@ -14,12 +14,12 @@ using ::testing::Exactly;
using ::testing::InvokeWithoutArgs;
using ::testing::NotNull;
-namespace {
+namespace media {
-const int kSampleRate = AudioParameters::kAudioCDSampleRate;
-const int kBitsPerSample = 16;
-const int kChannels = 2;
-const int kSamplesPerPacket = kSampleRate / 10;
+static const int kSampleRate = AudioParameters::kAudioCDSampleRate;
+static const int kBitsPerSample = 16;
+static const int kChannels = 2;
+static const int kSamplesPerPacket = kSampleRate / 10;
ACTION_P3(CheckCountAndSignalEvent, count, limit, event) {
if (++*count >= limit) {
@@ -27,11 +27,6 @@ ACTION_P3(CheckCountAndSignalEvent, count, limit, event) {
}
}
-// Test AudioInputController for create and close without recording audio.
Ami GONE FROM CHROMIUM 2011/03/04 22:52:04 Do you know if this was a TODO or what?
scherkus (not reviewing) 2011/03/07 14:57:56 Believe it corresponds with TEST(AudioInputControl
-}
-
-namespace media {
-
class MockAudioInputControllerEventHandler
: public AudioInputController::EventHandler {
public:

Powered by Google App Engine
This is Rietveld 408576698