Properly report number of quality downscales in stats.

webrtc/video/send_statistics_proxy.h

Index: webrtc/video/send_statistics_proxy.h
diff --git a/webrtc/video/send_statistics_proxy.h b/webrtc/video/send_statistics_proxy.h
index 934dab2f93e1f3af28d1f7f37084fcc1297b7760..7050141709f01940ce3e97959bacb2154b2b1aa7 100644
--- a/webrtc/video/send_statistics_proxy.h
+++ b/webrtc/video/send_statistics_proxy.h
@@ -58,8 +58,10 @@ class SendStatisticsProxy : public CpuOveruseMetricsObserver,
   void OnIncomingFrame(int width, int height);
 
   void OnCpuRestrictedResolutionChanged(bool cpu_restricted_resolution);
-  void OnQualityRestrictedResolutionChanged(bool restricted);
-  void SetResolutionRestrictionStats(bool bandwidth, bool cpu);
+  void OnQualityRestrictedResolutionChanged(int nr_of_downscales);

[stefan-webrtc, 2016/12/19 12:12:00]

s/nr_of_downscales/num_downscales, same for nr_of_quality_downscales. I think
that's a more common name for this type of thing.

Btw, should this say num_resolution_downscales?

+  void SetResolutionRestrictionStats(bool scaling_enabled,
+                                     bool cpu_restricted,
+                                     int nr_of_quality_downscales);
 
   void OnEncoderStatsUpdate(uint32_t framerate, uint32_t bitrate);
   void OnSuspendChange(bool is_suspended);
@@ -154,6 +156,7 @@ class SendStatisticsProxy : public CpuOveruseMetricsObserver,
   uint32_t last_sent_frame_timestamp_ GUARDED_BY(crit_);
   std::map<uint32_t, StatsUpdateTimes> update_times_ GUARDED_BY(crit_);
   rtc::ExpFilter encode_time_ GUARDED_BY(crit_);
+  int quality_downscales_ GUARDED_BY(crit_) = 0;

[åsapersson, 2016/12/16 14:45:14]

-1

[kthelgason, 2016/12/19 10:25:27]

Actually this makes sense as 0. When scaling is disabled this is set to -1
explicitly.

 
   // Contains stats used for UMA histograms. These stats will be reset if
   // content type changes between real-time video and screenshare, since these