Fix various MediaSource related crashes on Android.

media/base/android/media_source_player.cc

 // Copyright (c) 2013 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 "media/base/android/media_source_player.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 190 matching lines @@
       sampling_rate_(0),
       seekable_(true),
       audio_finished_(true),
       video_finished_(true),
       playing_(false),
       audio_access_unit_index_(0),
       video_access_unit_index_(0),
       waiting_for_audio_data_(false),
       waiting_for_video_data_(false),
       weak_this_(this) {
-  OnMediaMetadataChanged(duration_, width_, height_, false);

[qinmin, 2013/05/22 18:51:24]

thanks for doing this, i was about to upload a CL doing the same thing

 }
 
 MediaSourcePlayer::~MediaSourcePlayer() {
   Release();
 }
 
 void MediaSourcePlayer::SetVideoSurface(jobject surface) {
   video_decoder_job_.reset();
   if (!surface) {
     return;
   }
 
-  video_decoder_job_.reset(new VideoDecoderJob(
-      base::MessageLoopProxy::current(), video_codec_,
-      gfx::Size(width_, height_), surface));
+  if (HasVideo()) {
+    video_decoder_job_.reset(new VideoDecoderJob(
+        base::MessageLoopProxy::current(), video_codec_,
+        gfx::Size(width_, height_), surface));
+  }
 
   if (pending_play_)
     StartInternal();
 
   // Inform the fullscreen view the player is ready.
   // TODO(qinmin): refactor MediaPlayerBridge so that we have a better way
   // to inform ContentVideoView.
   OnMediaMetadataChanged(duration_, width_, height_, true);
 }
 
@@ skipping 215 matching lines @@
 
 bool MediaSourcePlayer::HasVideo() {
   return kUnknownVideoCodec != video_codec_;
 }
 
 bool MediaSourcePlayer::HasAudio() {
   return kUnknownAudioCodec != audio_codec_;
 }
 
 }  // namespace media