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

Unified Diff: content/renderer/media/renderer_webmediaplayer_delegate.h

Issue 2333983002: Reduce number of active codecs on low end devices. (Closed)
Patch Set: Created 4 years, 3 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: content/renderer/media/renderer_webmediaplayer_delegate.h
diff --git a/content/renderer/media/renderer_webmediaplayer_delegate.h b/content/renderer/media/renderer_webmediaplayer_delegate.h
index 001f7984a7c7350d51d5de24cda34c04c37214ac..3062cedd364dc7bca7762d1f1332d6ae0c8e82df 100644
--- a/content/renderer/media/renderer_webmediaplayer_delegate.h
+++ b/content/renderer/media/renderer_webmediaplayer_delegate.h
@@ -86,8 +86,9 @@ class CONTENT_EXPORT RendererWebMediaPlayerDelegate
void AddIdleDelegate(int delegate_id);
void RemoveIdleDelegate(int delegate_id);
- // Runs periodically to suspend idle delegates in |idle_delegate_map_|.
- void CleanupIdleDelegates();
+ // Runs periodically to suspend idle delegates in |idle_delegate_map_| which
+ // have been idle for longer than |idle_timeout|.
+ void CleanupIdleDelegates(base::TimeDelta idle_timeout);
// Setter for |is_playing_background_video_| that updates the metrics.
void SetIsPlayingBackgroundVideo(bool is_playing);
@@ -99,7 +100,7 @@ class CONTENT_EXPORT RendererWebMediaPlayerDelegate
// inactivity these players will be suspended to release unused resources.
bool idle_cleanup_running_ = false;
std::map<int, base::TimeTicks> idle_delegate_map_;
- base::RepeatingTimer idle_cleanup_timer_;
+ base::Timer idle_cleanup_timer_;
// Amount of time allowed to elapse after a delegate enters the paused before
// the delegate is suspended.

Powered by Google App Engine
This is Rietveld 408576698