Android: Propagate sample rate change to audio decoder job

media/base/android/media_decoder_job.cc

Index: media/base/android/media_decoder_job.cc
diff --git a/media/base/android/media_decoder_job.cc b/media/base/android/media_decoder_job.cc
index d96785e2121de96e6162c280da8ee3f6ef6dbdbf..a5b0da6f54c77c99b5752d503b8d7ef09d8de6a5 100644
--- a/media/base/android/media_decoder_job.cc
+++ b/media/base/android/media_decoder_job.cc
@@ -445,7 +445,7 @@ void MediaDecoderJob::DecodeInternal(
   // TODO(xhwang/qinmin): This logic is correct but strange. Clean it up.
   if (output_eos_encountered_)
     status = MEDIA_CODEC_OUTPUT_END_OF_STREAM;
-  else if (has_format_change)
+  else if (has_format_change && UpdateOutputFormat())

[gunsch, 2015/01/12 16:30:51]

What's the reason for moving UpdateOutputFormat up here?

[kjoswiak, 2015/01/12 19:44:48]

Short story, it crashes without this change.

Long story, when we get the format changed information, it means the buffers we
just got from decoder have new format. If this call is where it was previously,
in the OnDecodeComplete function, we only update the internal state after the
Decode process. However, we now need this information to be up to date during
the decode (specifically before we run ReleaseOutputBuffer, which is where we
render the buffers and do the timestamp helper stuff). Tracking these frames at
an incorrect rate, and then suddenly on the next call at the correct one,
happens to screw up the state enough to cause check failure. I think there's
technically another fix I could do to make the tracking more robust, but I think
its best we ensure we are always working with correct information =)

Also shouldn't break anything else, as https://codereview.chromium.org/730133003
almost reduced this function to a no-op video side, or at very least negated
what appears to be the original intent of this code path.

     status = MEDIA_CODEC_OUTPUT_FORMAT_CHANGED;
 
   bool render_output  = presentation_timestamp >= preroll_timestamp_ &&
@@ -543,8 +543,7 @@ void MediaDecoderJob::OnDecodeCompleted(
   }
 
   if (status == MEDIA_CODEC_OUTPUT_FORMAT_CHANGED) {
-    if (UpdateOutputFormat())
-      config_changed_cb_.Run();
+    config_changed_cb_.Run();
     status = MEDIA_CODEC_OK;
   }
 
@@ -649,10 +648,6 @@ bool MediaDecoderJob::IsCodecReconfigureNeeded(
   return true;
 }
 
-bool MediaDecoderJob::UpdateOutputFormat() {
-  return false;
-}
-
 void MediaDecoderJob::ReleaseMediaCodecBridge() {
   if (!media_codec_bridge_)
     return;