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

Side by Side Diff: webrtc/api/rtcstats_integrationtest.cc

Issue 2597963002: RTCIceCandidateStats.transportId added and collected. (Closed)
Patch Set: Rebase with master Created 3 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 unified diff | Download patch
« no previous file with comments | « no previous file | webrtc/api/rtcstatscollector.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright 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 372 matching lines...) Expand 10 before | Expand all | Expand 10 after
383 verifier.TestMemberIsNonNegative<uint64_t>( 383 verifier.TestMemberIsNonNegative<uint64_t>(
384 candidate_pair.consent_requests_sent); 384 candidate_pair.consent_requests_sent);
385 verifier.TestMemberIsUndefined(candidate_pair.consent_responses_received); 385 verifier.TestMemberIsUndefined(candidate_pair.consent_responses_received);
386 verifier.TestMemberIsUndefined(candidate_pair.consent_responses_sent); 386 verifier.TestMemberIsUndefined(candidate_pair.consent_responses_sent);
387 return verifier.ExpectAllMembersSuccessfullyTested(); 387 return verifier.ExpectAllMembersSuccessfullyTested();
388 } 388 }
389 389
390 bool VerifyRTCIceCandidateStats( 390 bool VerifyRTCIceCandidateStats(
391 const RTCIceCandidateStats& candidate) { 391 const RTCIceCandidateStats& candidate) {
392 RTCStatsVerifier verifier(report_, &candidate); 392 RTCStatsVerifier verifier(report_, &candidate);
393 verifier.TestMemberIsIDReference(
394 candidate.transport_id, RTCTransportStats::kType);
393 verifier.TestMemberIsDefined(candidate.is_remote); 395 verifier.TestMemberIsDefined(candidate.is_remote);
394 verifier.TestMemberIsDefined(candidate.ip); 396 verifier.TestMemberIsDefined(candidate.ip);
395 verifier.TestMemberIsNonNegative<int32_t>(candidate.port); 397 verifier.TestMemberIsNonNegative<int32_t>(candidate.port);
396 verifier.TestMemberIsDefined(candidate.protocol); 398 verifier.TestMemberIsDefined(candidate.protocol);
397 verifier.TestMemberIsDefined(candidate.candidate_type); 399 verifier.TestMemberIsDefined(candidate.candidate_type);
398 verifier.TestMemberIsNonNegative<int32_t>(candidate.priority); 400 verifier.TestMemberIsNonNegative<int32_t>(candidate.priority);
399 verifier.TestMemberIsUndefined(candidate.url); 401 verifier.TestMemberIsUndefined(candidate.url);
400 verifier.TestMemberIsDefined(candidate.deleted); 402 verifier.TestMemberIsDefined(candidate.deleted);
401 return verifier.ExpectAllMembersSuccessfullyTested(); 403 return verifier.ExpectAllMembersSuccessfullyTested();
402 } 404 }
(...skipping 206 matching lines...) Expand 10 before | Expand all | Expand 10 after
609 caller_ = nullptr; 611 caller_ = nullptr;
610 // Any pending stats requests should have completed in the act of destroying 612 // Any pending stats requests should have completed in the act of destroying
611 // the peer connection. 613 // the peer connection.
612 EXPECT_TRUE(stats_obtainer->report()); 614 EXPECT_TRUE(stats_obtainer->report());
613 } 615 }
614 #endif // HAVE_SCTP 616 #endif // HAVE_SCTP
615 617
616 } // namespace 618 } // namespace
617 619
618 } // namespace webrtc 620 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | webrtc/api/rtcstatscollector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698