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

Side by Side Diff: media/cast/sender/frame_sender.cc

Issue 562653004: Cast: First stab at implementing adaptive latency (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: refine formula, add comments Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #include "media/cast/sender/frame_sender.h" 5 #include "media/cast/sender/frame_sender.h"
6 6
7 #include "base/debug/trace_event.h" 7 #include "base/debug/trace_event.h"
8 8
9 namespace media { 9 namespace media {
10 namespace cast { 10 namespace cast {
11 namespace { 11 namespace {
12 12
13 const int kMinSchedulingDelayMs = 1; 13 const int kMinSchedulingDelayMs = 1;
14 const int kNumAggressiveReportsSentAtStart = 100; 14 const int kNumAggressiveReportsSentAtStart = 100;
15 15
16 } // namespace 16 } // namespace
17 17
18 FrameSender::FrameSender(scoped_refptr<CastEnvironment> cast_environment, 18 FrameSender::FrameSender(scoped_refptr<CastEnvironment> cast_environment,
19 bool is_audio, 19 bool is_audio,
20 CastTransportSender* const transport_sender, 20 CastTransportSender* const transport_sender,
21 base::TimeDelta rtcp_interval, 21 base::TimeDelta rtcp_interval,
22 int rtp_timebase, 22 int rtp_timebase,
23 uint32 ssrc, 23 uint32 ssrc,
24 double max_frame_rate, 24 double max_frame_rate,
25 base::TimeDelta playout_delay, 25 base::TimeDelta min_playout_delay,
26 base::TimeDelta max_playout_delay,
26 CongestionControl* congestion_control) 27 CongestionControl* congestion_control)
27 : cast_environment_(cast_environment), 28 : cast_environment_(cast_environment),
28 transport_sender_(transport_sender), 29 transport_sender_(transport_sender),
29 ssrc_(ssrc), 30 ssrc_(ssrc),
30 rtcp_interval_(rtcp_interval), 31 rtcp_interval_(rtcp_interval),
32 min_playout_delay_(min_playout_delay == base::TimeDelta() ?
33 max_playout_delay : min_playout_delay),
34 max_playout_delay_(max_playout_delay),
31 max_frame_rate_(max_frame_rate), 35 max_frame_rate_(max_frame_rate),
32 num_aggressive_rtcp_reports_sent_(0), 36 num_aggressive_rtcp_reports_sent_(0),
33 last_sent_frame_id_(0), 37 last_sent_frame_id_(0),
34 latest_acked_frame_id_(0), 38 latest_acked_frame_id_(0),
35 duplicate_ack_counter_(0), 39 duplicate_ack_counter_(0),
36 rtp_timebase_(rtp_timebase), 40 rtp_timebase_(rtp_timebase),
37 congestion_control_(congestion_control), 41 congestion_control_(congestion_control),
38 is_audio_(is_audio), 42 is_audio_(is_audio),
39 weak_factory_(this) { 43 weak_factory_(this) {
40 DCHECK(transport_sender_); 44 DCHECK(transport_sender_);
41 DCHECK_GT(rtp_timebase_, 0); 45 DCHECK_GT(rtp_timebase_, 0);
42 DCHECK(congestion_control_); 46 DCHECK(congestion_control_);
43 SetTargetPlayoutDelay(playout_delay); 47 SetTargetPlayoutDelay(min_playout_delay_);
44 send_target_playout_delay_ = false; 48 send_target_playout_delay_ = false;
45 memset(frame_rtp_timestamps_, 0, sizeof(frame_rtp_timestamps_)); 49 memset(frame_rtp_timestamps_, 0, sizeof(frame_rtp_timestamps_));
46 } 50 }
47 51
48 FrameSender::~FrameSender() { 52 FrameSender::~FrameSender() {
49 } 53 }
50 54
51 void FrameSender::ScheduleNextRtcpReport() { 55 void FrameSender::ScheduleNextRtcpReport() {
52 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN)); 56 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN));
53 base::TimeDelta time_to_next = rtcp_interval_; 57 base::TimeDelta time_to_next = rtcp_interval_;
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 ScheduleNextRtcpReport(); 92 ScheduleNextRtcpReport();
89 } 93 }
90 94
91 void FrameSender::OnMeasuredRoundTripTime(base::TimeDelta rtt) { 95 void FrameSender::OnMeasuredRoundTripTime(base::TimeDelta rtt) {
92 DCHECK(rtt > base::TimeDelta()); 96 DCHECK(rtt > base::TimeDelta());
93 current_round_trip_time_ = rtt; 97 current_round_trip_time_ = rtt;
94 } 98 }
95 99
96 void FrameSender::SetTargetPlayoutDelay( 100 void FrameSender::SetTargetPlayoutDelay(
97 base::TimeDelta new_target_playout_delay) { 101 base::TimeDelta new_target_playout_delay) {
102 new_target_playout_delay = std::max(new_target_playout_delay,
103 min_playout_delay_);
104 new_target_playout_delay = std::min(new_target_playout_delay,
105 max_playout_delay_);
98 target_playout_delay_ = new_target_playout_delay; 106 target_playout_delay_ = new_target_playout_delay;
99 max_unacked_frames_ = 107 max_unacked_frames_ =
100 std::min(kMaxUnackedFrames, 108 std::min(kMaxUnackedFrames,
101 1 + static_cast<int>(target_playout_delay_ * 109 1 + static_cast<int>(target_playout_delay_ *
102 max_frame_rate_ / 110 max_frame_rate_ /
103 base::TimeDelta::FromSeconds(1))); 111 base::TimeDelta::FromSeconds(1)));
104 send_target_playout_delay_ = true; 112 send_target_playout_delay_ = true;
105 } 113 }
106 114
107 void FrameSender::ResendCheck() { 115 void FrameSender::ResendCheck() {
(...skipping 211 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 << duration_in_flight.InMicroseconds() << " usec (" 327 << duration_in_flight.InMicroseconds() << " usec ("
320 << (target_playout_delay_ > base::TimeDelta() ? 328 << (target_playout_delay_ > base::TimeDelta() ?
321 100 * duration_in_flight / target_playout_delay_ : 329 100 * duration_in_flight / target_playout_delay_ :
322 kint64max) << "%)"; 330 kint64max) << "%)";
323 return frames_in_flight >= max_unacked_frames_ || 331 return frames_in_flight >= max_unacked_frames_ ||
324 duration_in_flight >= target_playout_delay_; 332 duration_in_flight >= target_playout_delay_;
325 } 333 }
326 334
327 } // namespace cast 335 } // namespace cast
328 } // namespace media 336 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698