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

Unified Diff: media/base/android/video_decoder_job.h

Issue 898843002: Use cached Key frames to avoid browser seek (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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/video_decoder_job.h
diff --git a/media/base/android/video_decoder_job.h b/media/base/android/video_decoder_job.h
index ea99a53373f6808f54f7c744e69647cd2f5f342c..0cba65d9f41a6360bf98301c386496baa0a25579 100644
--- a/media/base/android/video_decoder_job.h
+++ b/media/base/android/video_decoder_job.h
@@ -33,14 +33,9 @@ class VideoDecoderJob : public MediaDecoderJob {
// MediaDecoderJob implementation.
virtual bool HasStream() const override;
- virtual void Flush() override;
virtual void ReleaseDecoderResources() override;
virtual void SetDemuxerConfigs(const DemuxerConfigs& configs) override;
- bool next_video_data_is_iframe() {
- return next_video_data_is_iframe_;
- }
-
int output_width() const { return output_width_; }
int output_height() const { return output_height_; }
@@ -57,8 +52,7 @@ class VideoDecoderJob : public MediaDecoderJob {
const DemuxerConfigs& configs) const override;
virtual bool AreDemuxerConfigsChanged(
const DemuxerConfigs& configs) const override;
- virtual bool CreateMediaCodecBridgeInternal() override;
- virtual void CurrentDataConsumed(bool is_config_change) override;
+ virtual MediaDecoderJobStatus CreateMediaCodecBridgeInternal() override;
virtual bool UpdateOutputFormat() override;
// Returns true if a protected surface is required for video playback.
@@ -80,11 +74,6 @@ class VideoDecoderJob : public MediaDecoderJob {
base::Closure request_resources_cb_;
base::Closure release_resources_cb_;
- // Track whether the next access unit is an I-frame. The first access
- // unit after Flush() and CurrentDataConsumed(true) is guaranteed to be an
- // I-frame.
- bool next_video_data_is_iframe_;
-
DISALLOW_COPY_AND_ASSIGN(VideoDecoderJob);
};

Powered by Google App Engine
This is Rietveld 408576698