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

Unified Diff: media/filters/video_renderer_algorithm.h

Issue 2437623004: Fix perf and paint issues with VRI::PaintSingleFrame. (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « no previous file | media/filters/video_renderer_algorithm.cc » ('j') | media/filters/video_renderer_algorithm.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/video_renderer_algorithm.h
diff --git a/media/filters/video_renderer_algorithm.h b/media/filters/video_renderer_algorithm.h
index b5ce7932c223d29c156f6341b65f0f533f738a22..60eba421df54279be05930ab133acc5f5cc875b7 100644
--- a/media/filters/video_renderer_algorithm.h
+++ b/media/filters/video_renderer_algorithm.h
@@ -111,6 +111,9 @@ class MEDIA_EXPORT VideoRendererAlgorithm {
enum class ResetFlag { kEverything, kPreserveNextFrameEstimates };
void Reset(ResetFlag reset_flag = ResetFlag::kEverything);
+ // Returns the first frame in the frame queue and
chcunningham 2016/10/21 20:27:22 forgot to complete this guy
DaleCurtis 2016/10/21 21:51:17 Deleted.
+ scoped_refptr<VideoFrame> RenderFirstFrame();
chcunningham 2016/10/21 20:27:22 Not seeing this in the .cc?
DaleCurtis 2016/10/21 21:51:17 Deleted.
+
// Returns the number of frames currently buffered which could be rendered
// assuming current Render() interval trends.
//
@@ -136,8 +139,6 @@ class MEDIA_EXPORT VideoRendererAlgorithm {
size_t frames_queued() const { return frame_queue_.size(); }
- scoped_refptr<VideoFrame> first_frame() { return frame_queue_.front().frame; }
-
// Returns the average of the duration of all frames in |frame_queue_|
// as measured in wall clock (not media) time.
base::TimeDelta average_frame_duration() const {
« no previous file with comments | « no previous file | media/filters/video_renderer_algorithm.cc » ('j') | media/filters/video_renderer_algorithm.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698