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

Unified Diff: media/gpu/avda_shared_state.h

Issue 2461073002: Use MediaCodec.setOutputSurface() for fullscreen transitions on M. (Closed)
Patch Set: Simplify APIs. Created 4 years, 1 month 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/gpu/avda_shared_state.h
diff --git a/media/gpu/avda_shared_state.h b/media/gpu/avda_shared_state.h
index 843d4b7885cb8e14d504818e3be751456630481e..828b48856e56cf0c6588e0031a405f4adda49ebd 100644
--- a/media/gpu/avda_shared_state.h
+++ b/media/gpu/avda_shared_state.h
@@ -45,16 +45,6 @@ class AVDASharedState : public base::RefCounted<AVDASharedState> {
gl::GLSurface* surface() const { return surface_.get(); }
- // Iterates over all known codec images and updates the MediaCodec attached to
- // each one.
- void CodecChanged(MediaCodecBridge* codec);
-
- // Methods for finding and updating the AVDACodecImage associated with a given
- // picture buffer id. GetImageForPicture() will return null for unknown ids.
- // Calling SetImageForPicture() with a nullptr will erase the entry.
- void SetImageForPicture(int picture_buffer_id, AVDACodecImage* image);
- AVDACodecImage* GetImageForPicture(int picture_buffer_id) const;
-
// Helper method for coordinating the interactions between
// MediaCodec::ReleaseOutputBuffer() and WaitForFrameAvailable() when
// rendering to a SurfaceTexture; this method should never be called when
@@ -82,6 +72,8 @@ class AVDASharedState : public base::RefCounted<AVDASharedState> {
// StreamTextureMatrix contract. See GLStreamTextureImage::YInvertMatrix().
void GetTransformMatrix(float matrix[16]) const;
+ void clear_release_time() { release_time_ = base::TimeTicks(); }
+
protected:
virtual ~AVDASharedState();
@@ -112,6 +104,9 @@ class AVDASharedState : public base::RefCounted<AVDASharedState> {
// Texture matrix of the front buffer of the surface texture.
float gl_matrix_[16];
+ class OnFrameAvailableHandler;
+ scoped_refptr<OnFrameAvailableHandler> on_frame_available_handler_;
+
DISALLOW_COPY_AND_ASSIGN(AVDASharedState);
};

Powered by Google App Engine
This is Rietveld 408576698