Add QP stats to the statsview in AppRTCMobile for ios.

webrtc/examples/objc/AppRTCMobile/ARDStatsBuilder.m

 /*
  *  Copyright 2015 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 23 matching lines @@
   NSString *_videoEncodeMs;
   NSString *_videoInputFps;
   NSString *_videoInputHeight;
   NSString *_videoInputWidth;
   NSString *_videoSendCodec;
   NSString *_videoSendBitrate;
   NSString *_videoSendFps;
   NSString *_videoSendHeight;
   NSString *_videoSendWidth;
 
+  // QP stats.
+  int _videoQPSum;
+  int _framesEncoded;
+  int _oldVideoQPSum;
+  int _oldFramesEncoded;
+  int _deltaQPSum;
+  int _deltaFramesEncoded;

[kthelgason, 2016/12/16 09:41:37]

I don't think the _delta* variables need to be stored explicitly, as they can
always be computed from the other two. We should try to keep as little state as
possible IMO.

[daniela-webrtc, 2016/12/16 11:39:32]

Done.

+
   // Video receive stats.
   NSString *_videoDecodeMs;
   NSString *_videoDecodedFps;
   NSString *_videoOutputFps;
   NSString *_videoRecvBitrate;
   NSString *_videoRecvFps;
   NSString *_videoRecvHeight;
   NSString *_videoRecvWidth;
 
   // Audio send stats.
@@ skipping 16 matching lines @@
 }
 
 - (instancetype)init {
   if (self = [super init]) {
     _audioSendBitrateTracker = [[ARDBitrateTracker alloc] init];
     _audioRecvBitrateTracker = [[ARDBitrateTracker alloc] init];
     _connSendBitrateTracker = [[ARDBitrateTracker alloc] init];
     _connRecvBitrateTracker = [[ARDBitrateTracker alloc] init];
     _videoSendBitrateTracker = [[ARDBitrateTracker alloc] init];
     _videoRecvBitrateTracker = [[ARDBitrateTracker alloc] init];
+    _oldVideoQPSum = 0;
+    _oldFramesEncoded = 0;
   }
   return self;
 }
 
 - (NSString *)statsString {
   NSMutableString *result = [NSMutableString string];
   NSString *systemStatsFormat = @"(cpu)%ld%%\n";
   [result appendString:[NSString stringWithFormat:systemStatsFormat,
       (long)ARDGetCpuUsagePercentage()]];
 
   // Connection stats.
   NSString *connStatsFormat = @"CN %@ms | %@->%@/%@ | (s)%@ | (r)%@\n";
   [result appendString:[NSString stringWithFormat:connStatsFormat,
       _connRtt,
       _localCandType, _remoteCandType, _transportType,
       _connSendBitrate, _connRecvBitrate]];
 
   // Video send stats.
   NSString *videoSendFormat = @"VS (input) %@x%@@%@fps | (sent) %@x%@@%@fps\n"
-                               "VS (enc) %@/%@ | (sent) %@/%@ | %@ms | %@\n";
+                               "VS (enc) %@/%@ | (sent) %@/%@ | %@ms | %@\n"
+                               "QP %d | (QPSum)%d| (Frames encoded)%d\n ";

[kthelgason, 2016/12/16 09:41:37]

I'm not sure we ever care about the QPSum and frames encoded. Maybe we should
avoid cluttering up the HUD and just keep average QP? Also, please write
"average QP (last x frames)" or something.

[daniela-webrtc, 2016/12/16 11:39:32]

Done.

+  int avgqp = _deltaFramesEncoded ? _deltaQPSum / _deltaFramesEncoded  : 1;
   [result appendString:[NSString stringWithFormat:videoSendFormat,
       _videoInputWidth, _videoInputHeight, _videoInputFps,
       _videoSendWidth, _videoSendHeight, _videoSendFps,
       _actualEncBitrate, _targetEncBitrate,
       _videoSendBitrate, _availableSendBw,
       _videoEncodeMs,
-      _videoSendCodec]];
+      _videoSendCodec,
+      avgqp, _videoQPSum, _framesEncoded]];
 
   // Video receive stats.
   NSString *videoReceiveFormat =
       @"VR (recv) %@x%@@%@fps | (decoded)%@ | (output)%@fps | %@/%@ | %@ms\n";
   [result appendString:[NSString stringWithFormat:videoReceiveFormat,
       _videoRecvWidth, _videoRecvHeight, _videoRecvFps,
       _videoDecodedFps,
       _videoOutputFps,
       _videoRecvBitrate, _availableRecvBw,
       _videoDecodeMs]];
@@ skipping 117 matching lines @@
     } else if ([key isEqualToString:@"googFrameWidthSent"]) {
       _videoSendWidth = value;
     } else if ([key isEqualToString:@"googFrameRateSent"]) {
       _videoSendFps = value;
     } else if ([key isEqualToString:@"googAvgEncodeMs"]) {
       _videoEncodeMs = value;
     } else if ([key isEqualToString:@"bytesSent"]) {
       NSInteger byteCount = value.integerValue;
       [_videoSendBitrateTracker updateBitrateWithCurrentByteCount:byteCount];
       _videoSendBitrate = _videoSendBitrateTracker.bitrateString;
+    } else if ([key isEqualToString:@"qpSum"]) {
+      _videoQPSum = value.integerValue;
+      _deltaQPSum = _videoQPSum - _oldVideoQPSum;
+      _oldVideoQPSum = _videoQPSum;
+    } else if ([key isEqualToString:@"framesEncoded"]) {
+      _framesEncoded = value.integerValue;
+      _deltaFramesEncoded = _framesEncoded - _oldFramesEncoded;
+      _oldFramesEncoded = _framesEncoded;
     }
   }];
 }
 
 - (void)parseRecvSsrcStatsReport:(RTCLegacyStatsReport *)statsReport {
   NSDictionary *values = statsReport.values;
   if ([values objectForKey:@"googFrameWidthReceived"]) {
     // Video track.
     [self parseVideoRecvStatsReport:statsReport];
   } else if ([values objectForKey:@"audioOutputLevel"]) {
@@ skipping 37 matching lines @@
     } else if ([key isEqualToString:@"bytesReceived"]) {
       NSInteger byteCount = value.integerValue;
       [_videoRecvBitrateTracker updateBitrateWithCurrentByteCount:byteCount];
       _videoRecvBitrate = _videoRecvBitrateTracker.bitrateString;
     }
   }];
 }
 
 @end