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

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

Issue 1484403002: cast: Support for low-latency interactive mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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 <algorithm>
8 #include <vector>
9
7 #include "base/trace_event/trace_event.h" 10 #include "base/trace_event/trace_event.h"
8 #include "media/cast/cast_defines.h" 11 #include "media/cast/cast_defines.h"
9 #include "media/cast/constants.h" 12 #include "media/cast/constants.h"
10 #include "media/cast/sender/sender_encoded_frame.h" 13 #include "media/cast/sender/sender_encoded_frame.h"
11 14
12 namespace media { 15 namespace media {
13 namespace cast { 16 namespace cast {
14 namespace { 17 namespace {
15 18
16 const int kMinSchedulingDelayMs = 1; 19 const int kMinSchedulingDelayMs = 1;
17 const int kNumAggressiveReportsSentAtStart = 100; 20 const int kNumAggressiveReportsSentAtStart = 100;
18 21
19 // The additional number of frames that can be in-flight when input exceeds the 22 // The additional number of frames that can be in-flight when input exceeds the
20 // maximum frame rate. 23 // maximum frame rate.
21 const int kMaxFrameBurst = 5; 24 const int kMaxFrameBurst = 5;
22 25
23 } // namespace 26 } // namespace
24 27
25 // Convenience macro used in logging statements throughout this file. 28 // Convenience macro used in logging statements throughout this file.
26 #define SENDER_SSRC (is_audio_ ? "AUDIO[" : "VIDEO[") << ssrc_ << "] " 29 #define SENDER_SSRC (is_audio_ ? "AUDIO[" : "VIDEO[") << ssrc_ << "] "
27 30
28 FrameSender::FrameSender(scoped_refptr<CastEnvironment> cast_environment, 31 FrameSender::FrameSender(scoped_refptr<CastEnvironment> cast_environment,
29 bool is_audio, 32 bool is_audio,
30 CastTransportSender* const transport_sender, 33 CastTransportSender* const transport_sender,
31 int rtp_timebase, 34 int rtp_timebase,
32 uint32 ssrc, 35 uint32 ssrc,
33 double max_frame_rate, 36 double max_frame_rate,
34 base::TimeDelta min_playout_delay, 37 base::TimeDelta min_playout_delay,
35 base::TimeDelta max_playout_delay, 38 base::TimeDelta max_playout_delay,
39 base::TimeDelta animated_playout_delay,
36 CongestionControl* congestion_control) 40 CongestionControl* congestion_control)
37 : cast_environment_(cast_environment), 41 : cast_environment_(cast_environment),
38 transport_sender_(transport_sender), 42 transport_sender_(transport_sender),
39 ssrc_(ssrc), 43 ssrc_(ssrc),
40 min_playout_delay_(min_playout_delay == base::TimeDelta() ? 44 min_playout_delay_(min_playout_delay == base::TimeDelta()
41 max_playout_delay : min_playout_delay), 45 ? max_playout_delay
46 : min_playout_delay),
42 max_playout_delay_(max_playout_delay), 47 max_playout_delay_(max_playout_delay),
48 animated_playout_delay_(animated_playout_delay == base::TimeDelta()
49 ? max_playout_delay
50 : animated_playout_delay),
43 send_target_playout_delay_(false), 51 send_target_playout_delay_(false),
44 max_frame_rate_(max_frame_rate), 52 max_frame_rate_(max_frame_rate),
45 num_aggressive_rtcp_reports_sent_(0), 53 num_aggressive_rtcp_reports_sent_(0),
46 last_sent_frame_id_(0), 54 last_sent_frame_id_(0),
47 latest_acked_frame_id_(0), 55 latest_acked_frame_id_(0),
48 duplicate_ack_counter_(0), 56 duplicate_ack_counter_(0),
49 congestion_control_(congestion_control), 57 congestion_control_(congestion_control),
50 rtp_timebase_(rtp_timebase), 58 rtp_timebase_(rtp_timebase),
51 is_audio_(is_audio), 59 is_audio_(is_audio),
52 weak_factory_(this) { 60 weak_factory_(this) {
53 DCHECK(transport_sender_); 61 DCHECK(transport_sender_);
54 DCHECK_GT(rtp_timebase_, 0); 62 DCHECK_GT(rtp_timebase_, 0);
55 DCHECK(congestion_control_); 63 DCHECK(congestion_control_);
56 SetTargetPlayoutDelay(min_playout_delay_); 64 // We assume animated content to begin with since that is the common use
65 // case today.
66 VLOG(1) << "min latency " << min_playout_delay_.InMilliseconds()
miu 2015/12/03 21:26:23 nit: VLOG(1) << SENDER_SSRC << ...and the rest...
Irfan 2015/12/04 22:45:37 Done.
67 << "max latency " << max_playout_delay.InMilliseconds()
68 << "animated latency " << animated_playout_delay.InMilliseconds();
69 SetTargetPlayoutDelay(animated_playout_delay_);
57 send_target_playout_delay_ = false; 70 send_target_playout_delay_ = false;
58 memset(frame_rtp_timestamps_, 0, sizeof(frame_rtp_timestamps_)); 71 memset(frame_rtp_timestamps_, 0, sizeof(frame_rtp_timestamps_));
59 } 72 }
60 73
61 FrameSender::~FrameSender() { 74 FrameSender::~FrameSender() {
62 } 75 }
63 76
64 void FrameSender::ScheduleNextRtcpReport() { 77 void FrameSender::ScheduleNextRtcpReport() {
65 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN)); 78 DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::MAIN));
66 79
(...skipping 335 matching lines...) Expand 10 before | Expand all | Expand 10 after
402 VLOG(1) << SENDER_SSRC << "Dropping: In-flight duration would be too high."; 415 VLOG(1) << SENDER_SSRC << "Dropping: In-flight duration would be too high.";
403 return true; 416 return true;
404 } 417 }
405 418
406 // Next frame is accepted. 419 // Next frame is accepted.
407 return false; 420 return false;
408 } 421 }
409 422
410 } // namespace cast 423 } // namespace cast
411 } // namespace media 424 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698