Added support for multiple sampling rates on Windows.

remoting/host/audio_capturer.h

 // 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.
 
 #ifndef REMOTING_HOST_AUDIO_CAPTURER_H_
 #define REMOTING_HOST_AUDIO_CAPTURER_H_
 
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
-#include "remoting/proto/audio.pb.h"
 
 namespace remoting {
 
+class AudioPacket;
+
 class AudioCapturer {
  public:
   typedef base::Callback<void(scoped_ptr<AudioPacket> packet)>
       PacketCapturedCallback;
 
   virtual ~AudioCapturer() {}
 
   static scoped_ptr<AudioCapturer> Create();
 
   // Capturers should sample at a 44.1 kHz sampling rate, in uncompressed PCM

[Wez, 2012/07/25 09:23:48]

nit: This comment needs updating if AudioCapturer is going to support a variety
of rates.

[kxing, 2012/07/26 22:47:17]

Done.

   // stereo format. Capturers may choose the number of frames per packet.
   // Returns true on success.
   virtual bool Start(const PacketCapturedCallback& callback) = 0;
   // Stops the audio capturer, and frees the OS-specific audio capture
   // resources.
   virtual void Stop() = 0;
   // Returns true if the audio capturer is running.
   virtual bool IsRunning() = 0;
+
+  static bool IsValidSampleRate(int sample_rate);

[Wez, 2012/07/25 09:23:48]

Does it make sense for this to be a static? Won't the available sample rates
vary by AudioCapturer implementation? Or does this refer to the sample rates
that our protocol supports? Or that the current client implementation supports?

[kxing, 2012/07/26 22:47:17]

It represents the rates the current client implementation supports, so it should
be the same across all platforms.

 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_AUDIO_CAPTURER_H_