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

Side by Side Diff: webrtc/tools/event_log_visualizer/analyzer.cc

Issue 2353033005: Refactoring: move ownership of RtcEventLog from Call to PeerConnection (Closed)
Patch Set: Make TSan happy Created 4 years, 2 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 /* 1 /*
2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 934 matching lines...) Expand 10 before | Expand all | Expand 10 after
945 for (const auto& kv : rtcp_packets_) { 945 for (const auto& kv : rtcp_packets_) {
946 if (kv.first.GetDirection() == PacketDirection::kIncomingPacket) { 946 if (kv.first.GetDirection() == PacketDirection::kIncomingPacket) {
947 for (const LoggedRtcpPacket& rtcp_packet : kv.second) 947 for (const LoggedRtcpPacket& rtcp_packet : kv.second)
948 incoming_rtcp.insert( 948 incoming_rtcp.insert(
949 std::make_pair(rtcp_packet.timestamp, &rtcp_packet)); 949 std::make_pair(rtcp_packet.timestamp, &rtcp_packet));
950 } 950 }
951 } 951 }
952 952
953 SimulatedClock clock(0); 953 SimulatedClock clock(0);
954 BitrateObserver observer; 954 BitrateObserver observer;
955 RtcEventLogNullImpl null_event_log; 955 CongestionController cc(&clock, &observer, &observer, nullptr);
956 CongestionController cc(&clock, &observer, &observer, &null_event_log);
957 // TODO(holmer): Log the call config and use that here instead. 956 // TODO(holmer): Log the call config and use that here instead.
958 static const uint32_t kDefaultStartBitrateBps = 300000; 957 static const uint32_t kDefaultStartBitrateBps = 300000;
959 cc.SetBweBitrates(0, kDefaultStartBitrateBps, -1); 958 cc.SetBweBitrates(0, kDefaultStartBitrateBps, -1);
960 959
961 TimeSeries time_series; 960 TimeSeries time_series;
962 time_series.label = "Delay-based estimate"; 961 time_series.label = "Delay-based estimate";
963 time_series.style = LINE_DOT_GRAPH; 962 time_series.style = LINE_DOT_GRAPH;
964 TimeSeries acked_time_series; 963 TimeSeries acked_time_series;
965 acked_time_series.label = "Acked bitrate"; 964 acked_time_series.label = "Acked bitrate";
966 acked_time_series.style = LINE_DOT_GRAPH; 965 acked_time_series.style = LINE_DOT_GRAPH;
(...skipping 170 matching lines...) Expand 10 before | Expand all | Expand 10 after
1137 point.y -= estimated_base_delay_ms; 1136 point.y -= estimated_base_delay_ms;
1138 // Add the data set to the plot. 1137 // Add the data set to the plot.
1139 plot->series_list_.push_back(std::move(time_series)); 1138 plot->series_list_.push_back(std::move(time_series));
1140 1139
1141 plot->SetXAxis(0, call_duration_s_, "Time (s)", kLeftMargin, kRightMargin); 1140 plot->SetXAxis(0, call_duration_s_, "Time (s)", kLeftMargin, kRightMargin);
1142 plot->SetSuggestedYAxis(0, 10, "Delay (ms)", kBottomMargin, kTopMargin); 1141 plot->SetSuggestedYAxis(0, 10, "Delay (ms)", kBottomMargin, kTopMargin);
1143 plot->SetTitle("Network Delay Change."); 1142 plot->SetTitle("Network Delay Change.");
1144 } 1143 }
1145 } // namespace plotting 1144 } // namespace plotting
1146 } // namespace webrtc 1145 } // namespace webrtc
OLDNEW
« webrtc/DEPS ('K') | « webrtc/test/fuzzers/congestion_controller_feedback_fuzzer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698