Not streaming packets of silence.

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"
 
@@ skipping 14 matching lines @@
   // PCM 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);
+
+  static bool IsPacketOfSilence(const AudioPacket* packet);

[Sergey Ulanov, 2012/07/27 22:46:51]

This shouldn't to be part of AudioCapturer interface. At very least this
function should be marked as protected, as it is intended to be used by
interface implementations. But it would be better to put it to a separate file
audio_capturer_util.[h|cc] .

IsValidSampleRate() should be moved there too.

[kxing, 2012/07/30 16:55:18]

Done.

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