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

Unified Diff: media/cast/rtp_receiver/receiver_stats.cc

Issue 143263024: Cast: Refactoring RtpReceiver to Clang format (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/cast/rtp_receiver/receiver_stats.h ('k') | media/cast/rtp_receiver/receiver_stats_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/cast/rtp_receiver/receiver_stats.cc
diff --git a/media/cast/rtp_receiver/receiver_stats.cc b/media/cast/rtp_receiver/receiver_stats.cc
index 9d34583a769c185805c845d2c3a8848b7cfd925a..a9babaca37c91933eff95ecf0e76e3abfd16abb0 100644
--- a/media/cast/rtp_receiver/receiver_stats.cc
+++ b/media/cast/rtp_receiver/receiver_stats.cc
@@ -37,15 +37,16 @@ void ReceiverStats::GetStatistics(uint8* fraction_lost,
diff = max_sequence_number_ - interval_min_sequence_number_ + 1;
} else {
diff = kMaxSequenceNumber * (interval_wrap_count_ - 1) +
- (max_sequence_number_ - interval_min_sequence_number_ +
- kMaxSequenceNumber + 1);
+ (max_sequence_number_ - interval_min_sequence_number_ +
+ kMaxSequenceNumber + 1);
}
if (diff < 1) {
*fraction_lost = 0;
} else {
- *fraction_lost = static_cast<uint8>((256 * (1 -
- static_cast<float>(interval_number_packets_) / abs(diff))));
+ *fraction_lost = static_cast<uint8>(
+ (256 *
hubbe 2014/02/03 21:49:36 This is pretty ugly. Perhaps use fabs() and void t
mikhal1 2014/02/04 00:16:27 Done.
+ (1 - static_cast<float>(interval_number_packets_) / abs(diff))));
}
}
@@ -55,14 +56,15 @@ void ReceiverStats::GetStatistics(uint8* fraction_lost,
} else if (sequence_number_cycles_ == 0) {
*cumulative_lost = expected_packets_num - total_number_packets_;
} else {
- *cumulative_lost = kMaxSequenceNumber * (sequence_number_cycles_ - 1) +
+ *cumulative_lost =
+ kMaxSequenceNumber * (sequence_number_cycles_ - 1) +
(expected_packets_num - total_number_packets_ + kMaxSequenceNumber);
}
// Extended high sequence number consists of the highest seq number and the
// number of cycles (wrap).
- *extended_high_sequence_number = (sequence_number_cycles_ << 16) +
- max_sequence_number_;
+ *extended_high_sequence_number =
+ (sequence_number_cycles_ << 16) + max_sequence_number_;
*jitter = static_cast<uint32>(abs(jitter_.InMillisecondsRoundedUp()));
@@ -100,7 +102,8 @@ void ReceiverStats::UpdateStatistics(const RtpCastHeader& header) {
base::TimeDelta::FromMilliseconds(header.webrtc.header.timestamp);
if (total_number_packets_ > 0) {
// Update jitter.
- base::TimeDelta delta = (now - last_received_packet_time_) -
+ base::TimeDelta delta =
+ (now - last_received_packet_time_) -
((delta_new_timestamp - last_received_timestamp_) / 90);
jitter_ += (delta - jitter_) / 16;
}
« no previous file with comments | « media/cast/rtp_receiver/receiver_stats.h ('k') | media/cast/rtp_receiver/receiver_stats_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698