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

Unified Diff: talk/app/webrtc/statscollector_unittest.cc

Issue 1307633007: add mediaType field to ssrc stat (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: add tests Created 5 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 side-by-side diff with in-line comments
Download patch
Index: talk/app/webrtc/statscollector_unittest.cc
diff --git a/talk/app/webrtc/statscollector_unittest.cc b/talk/app/webrtc/statscollector_unittest.cc
index fd1ad617af4e49b2756c530e0be91746873b9246..00d9ccc4263c3aa3cce4c48e869e2893ccc46a6a 100644
--- a/talk/app/webrtc/statscollector_unittest.cc
+++ b/talk/app/webrtc/statscollector_unittest.cc
@@ -605,6 +605,10 @@ class StatsCollectorTest : public testing::Test {
*reports, StatsReport::kStatsValueNameSsrc);
EXPECT_EQ(rtc::ToString<uint32>(kSsrcOfTrack), ssrc_id);
+ std::string media_type = ExtractSsrcStatsValue(*reports,
+ StatsReport::kStatsValueNameMediaType);
+ EXPECT_EQ("audio", media_type);
+
// Verifies the values in the track report.
if (voice_sender_info) {
UpdateVoiceSenderInfoFromAudioTrack(audio_track, voice_sender_info);
@@ -1005,6 +1009,10 @@ TEST_F(StatsCollectorTest, TrackAndSsrcObjectExistAfterUpdateSsrcStats) {
std::string track_id = ExtractSsrcStatsValue(
reports, StatsReport::kStatsValueNameTrackId);
EXPECT_EQ(kLocalTrackId, track_id);
+
+ std::string media_type = ExtractSsrcStatsValue(reports,
+ StatsReport::kStatsValueNameMediaType);
+ EXPECT_EQ("video", media_type);
}
// This test verifies that an SSRC object has the identifier of a Transport

Powered by Google App Engine
This is Rietveld 408576698