Reports Prerender and Profile Total Bytes

chrome/browser/prerender/prerender_histograms.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/prerender/prerender_histograms.h"
 
 #include <string>
 
 #include "base/format_macros.h"
 #include "base/metrics/histogram.h"
@@ skipping 391 matching lines @@
 
 void PrerenderHistograms::RecordPrerenderPageVisitedStatus(
     Origin origin,
     uint8 experiment_id,
     bool visited_before) const {
   PREFIXED_HISTOGRAM_ORIGIN_EXPERIMENT(
       "PageVisitedStatus", origin, experiment_id,
       UMA_HISTOGRAM_BOOLEAN(name, visited_before));
 }
 
+void PrerenderHistograms::RecordNetworkBytes(bool used,
+                                             int64 prerender_bytes,
+                                             int64 profile_bytes) {
+  if (used) {
+    UMA_HISTOGRAM_COUNTS("Prerender.NetworkBytes.Used.KB",
+                         prerender_bytes >> 10);

[Alexei Svitkine (slow), 2014/02/13 15:50:50]

Wouldn't you be losing the remainder bytes each time you log this?

Perhaps it would be better to log the raw byte counts instead, rather than the
KB?

Alternatively, maybe you can keep a running total of the remainder that you add
to the value each time its logged (and update the new remainder)?

[jkarlin, 2014/02/13 16:04:14]

Done.

+  } else {
+    UMA_HISTOGRAM_COUNTS("Prerender.NetworkBytes.Wasted.KB",
+                         prerender_bytes >> 10);
+  }
+  UMA_HISTOGRAM_COUNTS("Prerender.NetworkBytes.TotalForProfile.KB",
+                       profile_bytes >> 10);
+}
+
 uint8 PrerenderHistograms::GetCurrentExperimentId() const {
   if (!WithinWindow())
     return kNoExperiment;
   return last_experiment_id_;
 }
 
 bool PrerenderHistograms::IsOriginExperimentWash() const {
   if (!WithinWindow())
     return false;
   return origin_experiment_wash_;
 }
 
 }  // namespace prerender