Simplified AudioOutputStream interface.

media/audio/fake_audio_input_stream.h

Index: media/audio/fake_audio_input_stream.h
diff --git a/media/audio/fake_audio_input_stream.h b/media/audio/fake_audio_input_stream.h
index d1e3260f1c21f4edc09b1bb5d616682323dbbcc8..62ab504c4ffe72b3138dec4aa93f97b2efa51c5b 100644
--- a/media/audio/fake_audio_input_stream.h
+++ b/media/audio/fake_audio_input_stream.h
@@ -20,8 +20,7 @@ class FakeAudioInputStream :
     public AudioInputStream,
     public base::RefCountedThreadSafe<FakeAudioInputStream> {
  public:
-  static AudioInputStream* MakeFakeStream(AudioParameters params,
-                                          int samples_per_packet);
+  static AudioInputStream* MakeFakeStream(AudioParameters params);

[scherkus (not reviewing), 2010/11/09 02:28:31]

I'm tempted to say make all these functions use const AudioParameters& params

(it's a whole 20 bytes being allocated on the stack!!!!!!!!!!!)

[Sergey Ulanov, 2010/11/09 22:29:58]

It is not any different from passing 5 parameters for this call, as we used to. 
There are many other places where AudioParameters are passed by value, so I
think it is better to fix this in a separate change. I've added todo in
audio_manager.h

 
   virtual bool Open();
   virtual void Start(AudioInputCallback* callback);
@@ -32,7 +31,7 @@ class FakeAudioInputStream :
   // Give RefCountedThreadSafe access our destructor.
   friend class base::RefCountedThreadSafe<FakeAudioInputStream>;
 
-  FakeAudioInputStream(AudioParameters params, int samples_per_packet);
+  FakeAudioInputStream(AudioParameters params);
   virtual ~FakeAudioInputStream();
 
   void DoCallback();