RTCMediaStreamTrackStats collected on a per-attachment basis.

webrtc/api/rtcstats_integrationtest.cc

 /*
  *  Copyright 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 415 matching lines @@
       const RTCMediaStreamTrackStats& media_stream_track) {
     RTCStatsVerifier verifier(report_, &media_stream_track);
     verifier.TestMemberIsDefined(media_stream_track.track_identifier);
     verifier.TestMemberIsDefined(media_stream_track.remote_source);
     verifier.TestMemberIsDefined(media_stream_track.ended);
     verifier.TestMemberIsDefined(media_stream_track.detached);
     verifier.TestMemberIsDefined(media_stream_track.kind);
     // Video or audio media stream track?
     if (*media_stream_track.kind == RTCMediaStreamTrackKind::kVideo) {
       // Video-only members
-      if (!*media_stream_track.remote_source) {
-        verifier.TestMemberIsNonNegative<uint32_t>(
-            media_stream_track.frame_width);
-        verifier.TestMemberIsNonNegative<uint32_t>(
-            media_stream_track.frame_height);
-      } else {
-        // TODO(hbos): Frame width/height should be defined for the remote case.
-        // Is this a real problem or an artifact of testing? crbug.com/659137
-        verifier.TestMemberIsUndefined(media_stream_track.frame_width);
-        verifier.TestMemberIsUndefined(media_stream_track.frame_height);
-      }
+      verifier.TestMemberIsNonNegative<uint32_t>(
+          media_stream_track.frame_width);
+      verifier.TestMemberIsNonNegative<uint32_t>(
+          media_stream_track.frame_height);
       verifier.TestMemberIsUndefined(media_stream_track.frames_per_second);
       verifier.TestMemberIsUndefined(media_stream_track.frames_sent);
       verifier.TestMemberIsUndefined(media_stream_track.frames_received);
       verifier.TestMemberIsUndefined(media_stream_track.frames_decoded);
       verifier.TestMemberIsUndefined(media_stream_track.frames_dropped);
       verifier.TestMemberIsUndefined(media_stream_track.frames_corrupted);
       verifier.TestMemberIsUndefined(media_stream_track.partial_frames_lost);
       verifier.TestMemberIsUndefined(media_stream_track.full_frames_lost);
       // Audio-only members should be undefined
       verifier.TestMemberIsUndefined(media_stream_track.audio_level);
       verifier.TestMemberIsUndefined(media_stream_track.echo_return_loss);
       verifier.TestMemberIsUndefined(
           media_stream_track.echo_return_loss_enhancement);
     } else {
       // Video-only members should be undefined
       verifier.TestMemberIsUndefined(media_stream_track.frame_width);
       verifier.TestMemberIsUndefined(media_stream_track.frame_height);
       verifier.TestMemberIsUndefined(media_stream_track.frames_per_second);
       verifier.TestMemberIsUndefined(media_stream_track.frames_sent);
       verifier.TestMemberIsUndefined(media_stream_track.frames_received);
       verifier.TestMemberIsUndefined(media_stream_track.frames_decoded);
       verifier.TestMemberIsUndefined(media_stream_track.frames_dropped);
       verifier.TestMemberIsUndefined(media_stream_track.frames_corrupted);
       verifier.TestMemberIsUndefined(media_stream_track.partial_frames_lost);
       verifier.TestMemberIsUndefined(media_stream_track.full_frames_lost);
       // Audio-only members
-      // TODO(hbos): Why are the audio track missing |audio_level|,
-      // |echo_return_loss| and |echo_return_loss_enhancement|? Is this a real
-      // problem or does it have to do with testing and not using real devices?
+      verifier.TestMemberIsNonNegative<double>(media_stream_track.audio_level);
+      // TODO(hbos): Find out why |echo_return_loss| and
+      // |echo_return_loss_enhancement| are undefined in the integration test.
+      // Is this a real problem or a test problem? Update test so that they are.
       // crbug.com/627816
-      verifier.MarkMemberTested(media_stream_track.audio_level, true);
-      verifier.MarkMemberTested(media_stream_track.echo_return_loss, true);
-      verifier.MarkMemberTested(
-          media_stream_track.echo_return_loss_enhancement, true);
+      verifier.TestMemberIsUndefined(media_stream_track.echo_return_loss);
+      verifier.TestMemberIsUndefined(
+          media_stream_track.echo_return_loss_enhancement);
     }
     return verifier.ExpectAllMembersSuccessfullyTested();
   }
 
   bool VerifyRTCPeerConnectionStats(
       const RTCPeerConnectionStats& peer_connection) {
     RTCStatsVerifier verifier(report_, &peer_connection);
     verifier.TestMemberIsNonNegative<uint32_t>(
         peer_connection.data_channels_opened);
     verifier.TestMemberIsNonNegative<uint32_t>(
@@ skipping 128 matching lines @@
   caller_ = nullptr;
   // Any pending stats requests should have completed in the act of destroying
   // the peer connection.
   EXPECT_TRUE(stats_obtainer->report());
 }
 #endif  // HAVE_SCTP
 
 }  // namespace
 
 }  // namespace webrtc