Android: Propagate sample rate change to audio decoder job

media/base/android/video_decoder_job.cc

 // Copyright 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/video_decoder_job.h"
 
 #include "base/bind.h"
 #include "base/lazy_instance.h"
 #include "base/threading/thread.h"
 #include "media/base/android/media_codec_bridge.h"
@@ skipping 135 matching lines @@
     return false;
 
   request_resources_cb_.Run();
   return true;
 }
 
 void VideoDecoderJob::CurrentDataConsumed(bool is_config_change) {
   next_video_data_is_iframe_ = is_config_change;
 }
 
-bool VideoDecoderJob::UpdateOutputFormat() {
+bool VideoDecoderJob::UpdateOutputFormat(
+    const base::Closure& config_changed_cb) {
   if (!media_codec_bridge_)
-    return false;
+    return true;
   int prev_output_width = output_width_;
   int prev_output_height = output_height_;
   // See b/18224769. The values reported from MediaCodecBridge::GetOutputFormat
   // correspond to the actual video frame size, but this is not necessarily the
   // size that should be output.
   output_width_ = config_width_;
   output_height_ = config_height_;
-  return (output_width_ != prev_output_width) ||
-      (output_height_ != prev_output_height);
+  if ((output_width_ != prev_output_width) ||
+      (output_height_ != prev_output_height))
+    config_changed_cb.Run();

[qinmin, 2015/01/21 02:04:13]

nit: {} required for if statement that spans multiple lines.

[kjoswiak, 2015/01/21 03:21:18]

Done.

+
+  return true;
 }
 
 bool VideoDecoderJob::IsProtectedSurfaceRequired() {
   return is_content_encrypted() && drm_bridge() &&
       drm_bridge()->IsProtectedSurfaceRequired();
 }
 
 }  // namespace media