Adding compressed bytes to DRP pingback

components/data_reduction_proxy/proto/pageload_metrics.proto

Index: components/data_reduction_proxy/proto/pageload_metrics.proto
diff --git a/components/data_reduction_proxy/proto/pageload_metrics.proto b/components/data_reduction_proxy/proto/pageload_metrics.proto
index cd9d1420b257f14a4008ccbede0afe2484091f05..764646a0f28dc4d27c5c8bcd710e72c73855f291 100644
--- a/components/data_reduction_proxy/proto/pageload_metrics.proto
+++ b/components/data_reduction_proxy/proto/pageload_metrics.proto
@@ -55,10 +55,10 @@ message PageloadMetrics {
 
   // The sum of original-content-length values, over resources that were not
   // loaded from browser cache.
-  optional int32 original_page_size_bytes = 10;
+  optional int32 original_page_size_kb = 10;

[tbansal1, 2017/01/19 22:01:18]

nitpicking: s/kb/kB/
https://en.wikipedia.org/wiki/Kilobyte says Kilobyte has symbol "kB"
https://en.wikipedia.org/wiki/Kilobit says kilobit has symbol "kb".

[RyanSturm, 2017/01/19 22:11:34]

Acknowledged. Contextually, this should be understandable, and protos are lower
case.

[tbansal1, 2017/01/19 22:24:09]

OK, I will leave up to you. There have been cases in the past where this has
caused confusion in the past (http://shortn/_wNyN3swrgZ,
http://shortn/_1faZ2biMVU). 

[RyanSturm, 2017/01/25 22:14:45]

obsolete

   // The sum of (compressed) content-length, over resources that were not loaded
   // from browser cache.
-  optional int32 compressed_page_size_bytes = 11;
+  optional int32 compressed_page_size_kb = 11;
 
   // The effective connection type at the start of the navigation.
   optional EffectiveConnectionType effective_connection_type = 12;