Added more error checking for audio packets.

remoting/client/audio_decode_scheduler.cc

Index: remoting/client/audio_decode_scheduler.cc
diff --git a/remoting/client/audio_decode_scheduler.cc b/remoting/client/audio_decode_scheduler.cc
index 7dc0d3cf717e4bfc7bef3b57cbe6c134520233ab..c5c2a0745e3a08a13b6bef6073ed3866cc93e43f 100644
--- a/remoting/client/audio_decode_scheduler.cc
+++ b/remoting/client/audio_decode_scheduler.cc
@@ -51,7 +51,9 @@ void AudioDecodeScheduler::DecodePacket(scoped_ptr<AudioPacket> packet,
 void AudioDecodeScheduler::ProcessDecodedPacket(scoped_ptr<AudioPacket> packet,
                                                 const base::Closure& done) {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
-  audio_player_->ProcessAudioPacket(packet.Pass());
+  // Only process packet if they are non-null.

[Wez, 2012/08/20 23:23:51]

nit: Only process |packet| if it is non-NULL.

[Wez, 2012/08/20 23:23:51]

Clarify under what circumstances |packet| can be NULL, either here, or on the
AudioDecoder interface's description of Decode().

[kxing, 2012/08/21 14:25:53]

Done.

[kxing, 2012/08/21 14:25:53]

Done.

+  if (packet.get())
+    audio_player_->ProcessAudioPacket(packet.Pass());
   done.Run();
 }