Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(836)

Unified Diff: media/base/android/media_codec_video_decoder.cc

Issue 1128383003: Implementation of MediaCodecPlayer stage 1 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Restored media_source_player_unittest.cc Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/android/media_codec_video_decoder.cc
diff --git a/media/base/android/media_codec_video_decoder.cc b/media/base/android/media_codec_video_decoder.cc
new file mode 100644
index 0000000000000000000000000000000000000000..4ccae8e7cfb7a837eaaecfedf2b32bb00d64c34d
--- /dev/null
+++ b/media/base/android/media_codec_video_decoder.cc
@@ -0,0 +1,256 @@
+// Copyright 2015 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_codec_video_decoder.h"
+
+#include "base/bind.h"
+#include "base/logging.h"
+#include "media/base/android/media_codec_bridge.h"
+#include "media/base/buffers.h"
+#include "media/base/demuxer_stream.h"
+
+namespace media {
+
+namespace {
+const int DELAY_FOR_STAND_ALONE_EOS = 2; // milliseconds
+}
+
+MediaCodecVideoDecoder::MediaCodecVideoDecoder(
+ const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
+ const base::Closure& request_data_cb,
+ const base::Closure& starvation_cb,
+ const base::Closure& stop_done_cb,
+ const base::Closure& error_cb,
+ const VideoSizeChangedCallback& video_size_changed_cb,
+ const base::Closure& codec_created_cb)
+ : MediaCodecDecoder(media_task_runner,
+ request_data_cb,
+ starvation_cb,
+ stop_done_cb,
+ error_cb,
+ "VideoDecoder"),
+ video_size_changed_cb_(video_size_changed_cb),
+ codec_created_cb_(codec_created_cb)
+{}
+
+MediaCodecVideoDecoder::~MediaCodecVideoDecoder()
+{
+ DVLOG(1) << "VideoDecoder::~VideoDecoder()";
watk 2015/06/04 20:56:43 Why aren't you using "MediaCodecVideoDecoder"? IMO
Tima Vaisburd 2015/06/05 04:17:47 It corresponds to what class_name() returns. I'd l
+}
+
+bool MediaCodecVideoDecoder::HasStream() const {
+ // Media thread
+ DCHECK(media_task_runner_->BelongsToCurrentThread());
+
+ return configs_.video_codec != kUnknownVideoCodec;
+}
+
+void MediaCodecVideoDecoder::SetDemuxerConfigs(const DemuxerConfigs& configs) {
+ // Media thread
+ DCHECK(media_task_runner_->BelongsToCurrentThread());
+
+ DVLOG(1) << class_name() << "::" << __FUNCTION__;
+
+ configs_ = configs;
+
+ if (video_size_.IsEmpty()) {
+ video_size_ = configs_.video_size;
+ media_task_runner_->PostTask(
+ FROM_HERE, base::Bind(video_size_changed_cb_, video_size_));
+ }
+}
+
+void MediaCodecVideoDecoder::ReleaseDecoderResources() {
+ // Media thread
+ DCHECK(media_task_runner_->BelongsToCurrentThread());
+
+ DVLOG(1) << class_name() << "::" << __FUNCTION__;
+
+ MediaCodecDecoder::ReleaseDecoderResources();
+ surface_ = gfx::ScopedJavaSurface();
+ delayed_buffers_.clear();
+}
+
+void MediaCodecVideoDecoder::SetPendingSurface(gfx::ScopedJavaSurface surface) {
+ // Media thread
+ DCHECK(media_task_runner_->BelongsToCurrentThread());
+
+ surface_ = surface.Pass();
+
+ if (surface_.IsEmpty()) {
+ // Synchronously stop decoder thread and release MediaCodec
+ ReleaseDecoderResources();
+ }
+}
+
+bool MediaCodecVideoDecoder::HasPendingSurface() const {
+ // Media thread
+ DCHECK(media_task_runner_->BelongsToCurrentThread());
+
+ return !surface_.IsEmpty();
+}
+
+bool MediaCodecVideoDecoder::IsCodecReconfigureNeeded(
+ const DemuxerConfigs& curr, const DemuxerConfigs& next) const {
+
+ if (curr.video_codec != next.video_codec ||
+ curr.is_video_encrypted != next.is_video_encrypted)
+ return true;
+
+ // Only size changes below this point
+
+ if (curr.video_size.width() == next.video_size.width() &&
+ curr.video_size.height() == next.video_size.height())
+ return false; // i.e. curr == next
+
+ return !static_cast<VideoCodecBridge*>(media_codec_bridge_.get())->
+ IsAdaptivePlaybackSupported(next.video_size.width(),
+ next.video_size.height());
+}
+
+MediaCodecDecoder::ConfigStatus
+MediaCodecVideoDecoder::ConfigureInternal() {
+ // Media thread
+ DCHECK(media_task_runner_->BelongsToCurrentThread());
+
+ DVLOG(1) << class_name() << "::" << __FUNCTION__;
+
+ // If we cannot find a key frame in cache, the browser seek is needed.
+ if (!au_queue_.SkipToKeyFrame()) {
+ DVLOG(1) << class_name() << "::" << __FUNCTION__ << " key frame required";
+
+ // The processing of CONFIG_KEY_FRAME_REQUIRED is not implemented yet,
+ // return error for now
+ //return CONFIG_KEY_FRAME_REQUIRED;
+ return CONFIG_FAILURE;
+ }
+
+ //bool is_secure = is_content_encrypted() && drm_bridge() &&
+ // drm_bridge()->IsProtectedSurfaceRequired();
+
+ bool is_secure = false; // DRM is not implemented
+
+ DCHECK(!surface_.IsEmpty());
+
+ media_codec_bridge_.reset(VideoCodecBridge::CreateDecoder(
+ configs_.video_codec,
+ is_secure,
+ configs_.video_size,
+ surface_.j_surface().obj(),
+ GetMediaCrypto().obj()));
+
+ if (!media_codec_bridge_) {
+ DVLOG(1) << class_name() << "::" << __FUNCTION__ << " failed";
+ return CONFIG_FAILURE;
+ }
+
+ DVLOG(1) << class_name() << "::" << __FUNCTION__ << " succeeded";
+
+ media_task_runner_->PostTask(FROM_HERE, codec_created_cb_);
+
+ return CONFIG_OK;
+}
+
+void MediaCodecVideoDecoder::SynchronizePTSWithTime(
+ base::TimeDelta current_time) {
+ // Media thread
+ DCHECK(media_task_runner_->BelongsToCurrentThread());
+
+ start_time_ticks_ = base::TimeTicks::Now();
+ start_pts_ = current_time;
+ last_seen_pts_ = current_time;
+}
+
+void MediaCodecVideoDecoder::OnOutputFormatChanged() {
+ // Decoder thread
+ DCHECK(decoder_thread_.task_runner()->BelongsToCurrentThread());
+
+ gfx::Size prev_size = video_size_;
+
+ // 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.
+ video_size_ = configs_.video_size;
+ if (video_size_ != prev_size)
+ media_task_runner_->PostTask(
+ FROM_HERE, base::Bind(video_size_changed_cb_, video_size_));
+}
+
+void MediaCodecVideoDecoder::Render(int buffer_index,
+ size_t size,
+ bool render_output,
+ base::TimeDelta pts,
+ bool eos_encountered) {
+ // Decoder thread
+ DCHECK(decoder_thread_.task_runner()->BelongsToCurrentThread());
+
+ DVLOG(2) << class_name() << "::" << __FUNCTION__ << " pts:" << pts
+ << " index:" << buffer_index << (eos_encountered ? " EOS" : "");
+
+ // EOS might come as a stand-alone frame with zero size
+ if (!size && eos_encountered) {
+ // Stand-alone EOS
+ // Discard the PTS that comes with it and ensure it is released last.
+ pts = last_seen_pts_ +
+ base::TimeDelta::FromMilliseconds(DELAY_FOR_STAND_ALONE_EOS);
+ } else {
+ // Keep track of last seen PTS
+ last_seen_pts_ = pts;
+ }
+
+ if (!render_output) {
+ ReleaseOutputBuffer(buffer_index, false, eos_encountered);
+ return;
+ }
+
+ base::TimeDelta time_to_render =
+ pts - (base::TimeTicks::Now() - start_time_ticks_ + start_pts_);
+
+ if (time_to_render < base::TimeDelta()) {
+ // Skip late frames
+ ReleaseOutputBuffer(buffer_index, false, eos_encountered);
+ return;
+ }
+
+ delayed_buffers_.insert(buffer_index);
+
+ bool do_render = size > 0;
+ decoder_thread_.task_runner()->PostDelayedTask(
+ FROM_HERE,
+ base::Bind(&MediaCodecVideoDecoder::ReleaseOutputBuffer,
+ base::Unretained(this),
+ buffer_index, do_render, eos_encountered),
+ time_to_render);
+}
+
+int MediaCodecVideoDecoder::NumDelayedRenderTasks() const {
+ // Decoder thread
+ DCHECK(decoder_thread_.task_runner()->BelongsToCurrentThread());
+
+ return delayed_buffers_.size();
+}
+
+void MediaCodecVideoDecoder::ReleaseDelayedBuffers() {
+ // Media thread
+ // Called when there is no decoder thread
+ for (int index : delayed_buffers_)
+ media_codec_bridge_->ReleaseOutputBuffer(index, false);
+ delayed_buffers_.clear();
+}
+
+void MediaCodecVideoDecoder::ReleaseOutputBuffer(
+ int buffer_index, bool render, bool eos_encountered) {
+ // Decoder thread
+ DCHECK(decoder_thread_.task_runner()->BelongsToCurrentThread());
+
+ DVLOG(2) << class_name() << "::" << __FUNCTION__;
+
+ media_codec_bridge_->ReleaseOutputBuffer(buffer_index, render);
+
+ delayed_buffers_.erase(buffer_index);
+
+ CheckLastFrame(eos_encountered, !delayed_buffers_.empty());
+}
+
+} // namespace media

Powered by Google App Engine
This is Rietveld 408576698