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

Side by Side Diff: media/cast/net/rtp/cast_message_builder.cc

Issue 540713002: Cast: Get rid of frame_id_map (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: removed unused variable 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/net/rtp/cast_message_builder.h" 5 #include "media/cast/net/rtp/cast_message_builder.h"
6 6
7 #include "media/cast/cast_defines.h" 7 #include "media/cast/cast_defines.h"
8 #include "media/cast/net/rtp/framer.h"
8 9
9 namespace media { 10 namespace media {
10 namespace cast { 11 namespace cast {
11 12
12 CastMessageBuilder::CastMessageBuilder( 13 CastMessageBuilder::CastMessageBuilder(
13 base::TickClock* clock, 14 base::TickClock* clock,
14 RtpPayloadFeedback* incoming_payload_feedback, 15 RtpPayloadFeedback* incoming_payload_feedback,
15 FrameIdMap* frame_id_map, 16 const Framer* framer,
16 uint32 media_ssrc, 17 uint32 media_ssrc,
17 bool decoder_faster_than_max_frame_rate, 18 bool decoder_faster_than_max_frame_rate,
18 int max_unacked_frames) 19 int max_unacked_frames)
19 : clock_(clock), 20 : clock_(clock),
20 cast_feedback_(incoming_payload_feedback), 21 cast_feedback_(incoming_payload_feedback),
21 frame_id_map_(frame_id_map), 22 framer_(framer),
22 media_ssrc_(media_ssrc), 23 media_ssrc_(media_ssrc),
23 decoder_faster_than_max_frame_rate_(decoder_faster_than_max_frame_rate), 24 decoder_faster_than_max_frame_rate_(decoder_faster_than_max_frame_rate),
24 max_unacked_frames_(max_unacked_frames), 25 max_unacked_frames_(max_unacked_frames),
25 cast_msg_(media_ssrc), 26 cast_msg_(media_ssrc),
26 slowing_down_ack_(false), 27 slowing_down_ack_(false),
27 acked_last_frame_(true), 28 acked_last_frame_(true),
28 last_acked_frame_id_(kStartFrameId) { 29 last_acked_frame_id_(kStartFrameId) {
29 cast_msg_.ack_frame_id = kStartFrameId; 30 cast_msg_.ack_frame_id = kStartFrameId;
30 } 31 }
31 32
(...skipping 12 matching lines...) Expand all
44 } 45 }
45 BuildPacketList(); 46 BuildPacketList();
46 47
47 // Send cast message. 48 // Send cast message.
48 VLOG(2) << "Send cast message Ack:" << static_cast<int>(frame_id); 49 VLOG(2) << "Send cast message Ack:" << static_cast<int>(frame_id);
49 cast_feedback_->CastFeedback(cast_msg_); 50 cast_feedback_->CastFeedback(cast_msg_);
50 } 51 }
51 52
52 bool CastMessageBuilder::UpdateAckMessage(uint32 frame_id) { 53 bool CastMessageBuilder::UpdateAckMessage(uint32 frame_id) {
53 if (!decoder_faster_than_max_frame_rate_) { 54 if (!decoder_faster_than_max_frame_rate_) {
54 int complete_frame_count = frame_id_map_->NumberOfCompleteFrames(); 55 int complete_frame_count = framer_->NumberOfCompleteFrames();
55 if (complete_frame_count > max_unacked_frames_) { 56 if (complete_frame_count > max_unacked_frames_) {
56 // We have too many frames pending in our framer; slow down ACK. 57 // We have too many frames pending in our framer; slow down ACK.
57 if (!slowing_down_ack_) { 58 if (!slowing_down_ack_) {
58 slowing_down_ack_ = true; 59 slowing_down_ack_ = true;
59 ack_queue_.push_back(last_acked_frame_id_); 60 ack_queue_.push_back(last_acked_frame_id_);
60 } 61 }
61 } else if (complete_frame_count <= 1) { 62 } else if (complete_frame_count <= 1) {
62 // We are down to one or less frames in our framer; ACK normally. 63 // We are down to one or less frames in our framer; ACK normally.
63 slowing_down_ack_ = false; 64 slowing_down_ack_ = false;
64 ack_queue_.clear(); 65 ack_queue_.clear();
(...skipping 23 matching lines...) Expand all
88 last_acked_frame_id_ = frame_id; 89 last_acked_frame_id_ = frame_id;
89 cast_msg_.ack_frame_id = last_acked_frame_id_; 90 cast_msg_.ack_frame_id = last_acked_frame_id_;
90 cast_msg_.missing_frames_and_packets.clear(); 91 cast_msg_.missing_frames_and_packets.clear();
91 last_update_time_ = clock_->NowTicks(); 92 last_update_time_ = clock_->NowTicks();
92 return true; 93 return true;
93 } 94 }
94 95
95 bool CastMessageBuilder::TimeToSendNextCastMessage( 96 bool CastMessageBuilder::TimeToSendNextCastMessage(
96 base::TimeTicks* time_to_send) { 97 base::TimeTicks* time_to_send) {
97 // We haven't received any packets. 98 // We haven't received any packets.
98 if (last_update_time_.is_null() && frame_id_map_->Empty()) 99 if (last_update_time_.is_null() && framer_->Empty())
99 return false; 100 return false;
100 101
101 *time_to_send = last_update_time_ + base::TimeDelta::FromMilliseconds( 102 *time_to_send = last_update_time_ + base::TimeDelta::FromMilliseconds(
102 kCastMessageUpdateIntervalMs); 103 kCastMessageUpdateIntervalMs);
103 return true; 104 return true;
104 } 105 }
105 106
106 void CastMessageBuilder::UpdateCastMessage() { 107 void CastMessageBuilder::UpdateCastMessage() {
107 RtcpCastMessage message(media_ssrc_); 108 RtcpCastMessage message(media_ssrc_);
108 if (!UpdateCastMessageInternal(&message)) 109 if (!UpdateCastMessageInternal(&message))
109 return; 110 return;
110 111
111 // Send cast message. 112 // Send cast message.
112 cast_feedback_->CastFeedback(message); 113 cast_feedback_->CastFeedback(message);
113 } 114 }
114 115
115 void CastMessageBuilder::Reset() { 116 void CastMessageBuilder::Reset() {
116 cast_msg_.ack_frame_id = kStartFrameId; 117 cast_msg_.ack_frame_id = kStartFrameId;
117 cast_msg_.missing_frames_and_packets.clear(); 118 cast_msg_.missing_frames_and_packets.clear();
118 time_last_nacked_map_.clear(); 119 time_last_nacked_map_.clear();
119 } 120 }
120 121
121 bool CastMessageBuilder::UpdateCastMessageInternal(RtcpCastMessage* message) { 122 bool CastMessageBuilder::UpdateCastMessageInternal(RtcpCastMessage* message) {
122 if (last_update_time_.is_null()) { 123 if (last_update_time_.is_null()) {
123 if (!frame_id_map_->Empty()) { 124 if (!framer_->Empty()) {
124 // We have received packets. 125 // We have received packets.
125 last_update_time_ = clock_->NowTicks(); 126 last_update_time_ = clock_->NowTicks();
126 } 127 }
127 return false; 128 return false;
128 } 129 }
129 // Is it time to update the cast message? 130 // Is it time to update the cast message?
130 base::TimeTicks now = clock_->NowTicks(); 131 base::TimeTicks now = clock_->NowTicks();
131 if (now - last_update_time_ < 132 if (now - last_update_time_ <
132 base::TimeDelta::FromMilliseconds(kCastMessageUpdateIntervalMs)) { 133 base::TimeDelta::FromMilliseconds(kCastMessageUpdateIntervalMs)) {
133 return false; 134 return false;
134 } 135 }
135 last_update_time_ = now; 136 last_update_time_ = now;
136 137
137 // Needed to cover when a frame is skipped. 138 // Needed to cover when a frame is skipped.
138 UpdateAckMessage(last_acked_frame_id_); 139 UpdateAckMessage(last_acked_frame_id_);
139 BuildPacketList(); 140 BuildPacketList();
140 *message = cast_msg_; 141 *message = cast_msg_;
141 return true; 142 return true;
142 } 143 }
143 144
144 void CastMessageBuilder::BuildPacketList() { 145 void CastMessageBuilder::BuildPacketList() {
145 base::TimeTicks now = clock_->NowTicks(); 146 base::TimeTicks now = clock_->NowTicks();
146 147
147 // Clear message NACK list. 148 // Clear message NACK list.
148 cast_msg_.missing_frames_and_packets.clear(); 149 cast_msg_.missing_frames_and_packets.clear();
149 150
150 // Are we missing packets? 151 // Are we missing packets?
151 if (frame_id_map_->Empty()) 152 if (framer_->Empty())
152 return; 153 return;
153 154
154 uint32 newest_frame_id = frame_id_map_->NewestFrameId(); 155 uint32 newest_frame_id = framer_->NewestFrameId();
155 uint32 next_expected_frame_id = cast_msg_.ack_frame_id + 1; 156 uint32 next_expected_frame_id = cast_msg_.ack_frame_id + 1;
156 157
157 // Iterate over all frames. 158 // Iterate over all frames.
158 for (; !IsNewerFrameId(next_expected_frame_id, newest_frame_id); 159 for (; !IsNewerFrameId(next_expected_frame_id, newest_frame_id);
159 ++next_expected_frame_id) { 160 ++next_expected_frame_id) {
160 TimeLastNackMap::iterator it = 161 TimeLastNackMap::iterator it =
161 time_last_nacked_map_.find(next_expected_frame_id); 162 time_last_nacked_map_.find(next_expected_frame_id);
162 if (it != time_last_nacked_map_.end()) { 163 if (it != time_last_nacked_map_.end()) {
163 // We have sent a NACK in this frame before, make sure enough time have 164 // We have sent a NACK in this frame before, make sure enough time have
164 // passed. 165 // passed.
165 if (now - it->second < 166 if (now - it->second <
166 base::TimeDelta::FromMilliseconds(kNackRepeatIntervalMs)) { 167 base::TimeDelta::FromMilliseconds(kNackRepeatIntervalMs)) {
167 continue; 168 continue;
168 } 169 }
169 } 170 }
170 171
171 PacketIdSet missing; 172 PacketIdSet missing;
172 if (frame_id_map_->FrameExists(next_expected_frame_id)) { 173 if (framer_->FrameExists(next_expected_frame_id)) {
173 bool last_frame = (newest_frame_id == next_expected_frame_id); 174 bool last_frame = (newest_frame_id == next_expected_frame_id);
174 frame_id_map_->GetMissingPackets( 175 framer_->GetMissingPackets(
175 next_expected_frame_id, last_frame, &missing); 176 next_expected_frame_id, last_frame, &missing);
176 if (!missing.empty()) { 177 if (!missing.empty()) {
177 time_last_nacked_map_[next_expected_frame_id] = now; 178 time_last_nacked_map_[next_expected_frame_id] = now;
178 cast_msg_.missing_frames_and_packets.insert( 179 cast_msg_.missing_frames_and_packets.insert(
179 std::make_pair(next_expected_frame_id, missing)); 180 std::make_pair(next_expected_frame_id, missing));
180 } 181 }
181 } else { 182 } else {
182 time_last_nacked_map_[next_expected_frame_id] = now; 183 time_last_nacked_map_[next_expected_frame_id] = now;
183 missing.insert(kRtcpCastAllPacketsLost); 184 missing.insert(kRtcpCastAllPacketsLost);
184 cast_msg_.missing_frames_and_packets[next_expected_frame_id] = missing; 185 cast_msg_.missing_frames_and_packets[next_expected_frame_id] = missing;
185 } 186 }
186 } 187 }
187 } 188 }
188 189
189 } // namespace cast 190 } // namespace cast
190 } // namespace media 191 } // namespace media
OLDNEW
« no previous file with comments | « media/cast/net/rtp/cast_message_builder.h ('k') | media/cast/net/rtp/cast_message_builder_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698