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

Side by Side Diff: media/remoting/remote_renderer_impl.h

Issue 2638393005: Media Remoting: Exit remoting only if playback is continuously delayed. (Closed)
Patch Set: Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef MEDIA_REMOTING_REMOTE_RENDERER_IMPL_H_ 5 #ifndef MEDIA_REMOTING_REMOTE_RENDERER_IMPL_H_
6 #define MEDIA_REMOTING_REMOTE_RENDERER_IMPL_H_ 6 #define MEDIA_REMOTING_REMOTE_RENDERER_IMPL_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 203 matching lines...) Expand 10 before | Expand all | Expand 10 after
214 214
215 // Stores all updates on the number of video frames decoded/dropped, and the 215 // Stores all updates on the number of video frames decoded/dropped, and the
216 // local time when updated in the moving time window. This is used to check 216 // local time when updated in the moving time window. This is used to check
217 // whether too many video frames were dropped. 217 // whether too many video frames were dropped.
218 std::deque<std::tuple<base::TimeTicks, int, int>> video_stats_queue_; 218 std::deque<std::tuple<base::TimeTicks, int, int>> video_stats_queue_;
219 219
220 // The total number of frames decoded/dropped in the time window. 220 // The total number of frames decoded/dropped in the time window.
221 int sum_video_frames_decoded_ = 0; 221 int sum_video_frames_decoded_ = 0;
222 int sum_video_frames_dropped_ = 0; 222 int sum_video_frames_dropped_ = 0;
223 223
224 // Records the number that remoting playback is continuously delayed.
miu 2017/01/19 22:00:55 s/number that/number of consecutive times/
miu 2017/01/19 22:00:55 s/is/was/
xjz 2017/01/19 22:10:04 Done.
xjz 2017/01/19 22:10:04 Done.
225 int num_playback_delayed_ = 0;
miu 2017/01/19 22:00:55 How about |times_playback_delayed_|?
xjz 2017/01/19 22:10:04 Done.
226
224 // Records events and measurements of interest. 227 // Records events and measurements of interest.
225 remoting::RendererMetricsRecorder metrics_recorder_; 228 remoting::RendererMetricsRecorder metrics_recorder_;
226 229
227 // A timer that polls the RemoteDemuxerStreamAdapters periodically to measure 230 // A timer that polls the RemoteDemuxerStreamAdapters periodically to measure
228 // the data flow rates for metrics. 231 // the data flow rates for metrics.
229 base::RepeatingTimer data_flow_poll_timer_; 232 base::RepeatingTimer data_flow_poll_timer_;
230 233
231 base::WeakPtrFactory<RemoteRendererImpl> weak_factory_; 234 base::WeakPtrFactory<RemoteRendererImpl> weak_factory_;
232 235
233 DISALLOW_COPY_AND_ASSIGN(RemoteRendererImpl); 236 DISALLOW_COPY_AND_ASSIGN(RemoteRendererImpl);
234 }; 237 };
235 238
236 } // namespace media 239 } // namespace media
237 240
238 #endif // MEDIA_REMOTING_REMOTE_RENDERER_IMPL_H_ 241 #endif // MEDIA_REMOTING_REMOTE_RENDERER_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | media/remoting/remote_renderer_impl.cc » ('j') | media/remoting/remote_renderer_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698