Added more error checking for audio packets.

remoting/host/audio_capturer_win.cc

 // 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.
 
 #include <windows.h>
 #include <audioclient.h>
 #include <avrt.h>
 #include <mmdeviceapi.h>
 #include <mmreg.h>
 #include <mmsystem.h>
@@ skipping 276 matching lines @@
       LOG(ERROR) << "Failed to GetBuffer. Error " << hr;
       return;
     }
 
     if (!IsPacketOfSilence(
             reinterpret_cast<const int16*>(data),
             frames * kChannels)) {
       scoped_ptr<AudioPacket> packet =
           scoped_ptr<AudioPacket>(new AudioPacket());
       packet->add_data(data, frames * wave_format_ex_->nBlockAlign);
+      packet->set_encoding(AudioPacket::ENCODING_RAW);
       packet->set_sampling_rate(sampling_rate_);
-      packet->set_bytes_per_sample(
-          static_cast<AudioPacket::BytesPerSample>(sizeof(int16)));
-      packet->set_encoding(AudioPacket::ENCODING_RAW);
+      packet->set_bytes_per_sample(AudioPacket::BYTES_PER_SAMPLE_2);
+      packet->set_channels(AudioPacket::CHANNELS_STEREO);
 
       callback_.Run(packet.Pass());
     }
 
     hr = audio_capture_client_->ReleaseBuffer(frames);
     if (FAILED(hr)) {
       LOG(ERROR) << "Failed to ReleaseBuffer. Error " << hr;
       return;
     }
   }
 }
 
 // Detects whether there is audio playing in a packet of samples.
 // Windows can give nonzero samples, even when there is no audio playing, so
 // extremely low amplitude samples are counted as silence.
 bool AudioCapturerWin::IsPacketOfSilence(
     const int16* samples, int number_of_samples) {
   for (int i = 0; i < number_of_samples; i++) {
     if (abs(samples[i]) > kSilenceThreshold)
       return false;
   }
   return true;
 }
 
 scoped_ptr<AudioCapturer> AudioCapturer::Create() {
   return scoped_ptr<AudioCapturer>(new AudioCapturerWin());
 }
 
 }  // namespace remoting