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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: media/audio/simple_sources_unittest.cc

Issue 8818012: Remove the AudioManager singleton. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Address comments from Avi Created 9 years 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
« media/audio/audio_output_proxy.cc ('K') | « media/audio/mac/audio_output_mac_unittest.cc ('k') | media/audio/test_audio_input_controller_factory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/simple_sources_unittest.cc
===================================================================
--- media/audio/simple_sources_unittest.cc (revision 113173)
+++ media/audio/simple_sources_unittest.cc (working copy)
@@ -21,7 +21,7 @@
}
for (uint32 i = 0; i < len; i++)
- buffer[i] = static_cast<char>(rand());
+ buffer[i] = static_cast<char>(rand()); // NOLINT
}
// To test write size smaller than read size.
@@ -68,8 +68,7 @@
SineWaveAudioSource source(SineWaveAudioSource::FORMAT_16BIT_LINEAR_PCM, 1,
freq, AudioParameters::kTelephoneSampleRate);
- AudioManager* audio_man = AudioManager::GetAudioManager();
- ASSERT_TRUE(NULL != audio_man);
+ scoped_refptr<AudioManager> audio_man(AudioManager::Create());
AudioParameters params(
AudioParameters::AUDIO_MOCK, CHANNEL_LAYOUT_MONO,
AudioParameters::kTelephoneSampleRate, bytes_per_sample * 2, samples);
« media/audio/audio_output_proxy.cc ('K') | « media/audio/mac/audio_output_mac_unittest.cc ('k') | media/audio/test_audio_input_controller_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698