Renamed fields in common_types.h/RtcpStatistics.

webrtc/common_types.h

Index: webrtc/common_types.h
diff --git a/webrtc/common_types.h b/webrtc/common_types.h
index 43ae642375d8d0e9f22acabf9f1684a125f0b25f..daaf1ff9732c9ce414eec621c0993795f2cea309 100644
--- a/webrtc/common_types.h
+++ b/webrtc/common_types.h
@@ -22,6 +22,7 @@
 #include "webrtc/api/video/video_timing.h"
 #include "webrtc/rtc_base/array_view.h"
 #include "webrtc/rtc_base/checks.h"
+#include "webrtc/rtc_base/deprecation.h"
 #include "webrtc/rtc_base/optional.h"
 #include "webrtc/typedefs.h"
 
@@ -156,13 +157,19 @@ enum FrameType {
 struct RtcpStatistics {
   RtcpStatistics()
       : fraction_lost(0),
-        cumulative_lost(0),
-        extended_max_sequence_number(0),
+        packets_lost(0),
+        extended_highest_sequence_number(0),
         jitter(0) {}
 
   uint8_t fraction_lost;
-  uint32_t cumulative_lost;
-  uint32_t extended_max_sequence_number;
+  union {
+    uint32_t packets_lost;
+    RTC_DEPRECATED uint32_t cumulative_lost;
+  };

[kwiberg-webrtc, 2017/08/01 08:26:51]

Hmm, interesting. This actually appears to *not* be undefined behavior, because
the union members are standard-layout types whose common subsequence covers all
of them. I'll have to remember this trick. :-)

+  union {
+    uint32_t extended_highest_sequence_number;
+    RTC_DEPRECATED uint32_t extended_max_sequence_number;
+  };
   uint32_t jitter;
 };