Domain Reliability: Add more upload-related histograms.

components/domain_reliability/context.cc

 // Copyright 2014 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 "components/domain_reliability/context.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 49 matching lines @@
       dispatcher_(dispatcher),
       uploader_(uploader),
       uploading_beacons_size_(0),
       last_network_change_time_(last_network_change_time),
       weak_factory_(this) {}
 
 DomainReliabilityContext::~DomainReliabilityContext() {
   ClearBeacons();
 }
 
-void DomainReliabilityContext::OnBeacon(
+bool DomainReliabilityContext::OnBeacon(
     std::unique_ptr<DomainReliabilityBeacon> beacon) {
   bool success = (beacon->status == "ok");
   double sample_rate = beacon->details.quic_port_migration_detected
                            ? 1.0
                            : config().GetSampleRate(success);
   bool should_report = base::RandDouble() < sample_rate;
   UMA_HISTOGRAM_BOOLEAN("DomainReliability.BeaconReported", should_report);
   if (!should_report) {
     // If the beacon isn't queued to be reported, it definitely cannot evict
     // an older beacon. (This histogram is also logged below based on whether
     // an older beacon was actually evicted.)
     LogOnBeaconDidEvictHistogram(false);
-    return;
+    return false;
   }
   beacon->sample_rate = sample_rate;
 
   UMA_HISTOGRAM_SPARSE_SLOWLY("DomainReliability.ReportedBeaconError",
                               -beacon->chrome_error);
   if (!beacon->server_ip.empty()) {
     UMA_HISTOGRAM_SPARSE_SLOWLY(
         "DomainReliability.ReportedBeaconError_HasServerIP",
         -beacon->chrome_error);
   }
+  UMA_HISTOGRAM_COUNTS_100("DomainReliability.ReportedBeaconUploadDepth",
+                           beacon->upload_depth);
   // TODO(juliatuttle): Histogram HTTP response code?
 
   // Allow beacons about reports, but don't schedule an upload for more than
   // one layer of recursion, to avoid infinite report loops.
   if (beacon->upload_depth <= kMaxUploadDepthToSchedule)
     scheduler_.OnBeaconAdded();
   beacons_.push_back(std::move(beacon));
   bool should_evict = beacons_.size() > kMaxQueuedBeacons;
   if (should_evict)
     RemoveOldestBeacon();
 
   LogOnBeaconDidEvictHistogram(should_evict);
+
+  base::TimeTicks now = base::TimeTicks::Now();
+  if (last_queued_beacon_time_ != base::TimeTicks()) {
+    UMA_HISTOGRAM_LONG_TIMES("DomainReliability.BeaconInterval",
+                             now - last_queued_beacon_time_);
+  }
+  last_queued_beacon_time_ = now;
+
+  return true;
 }
 
 void DomainReliabilityContext::ClearBeacons() {
   beacons_.clear();
   uploading_beacons_size_ = 0;
 }
 
 std::unique_ptr<Value> DomainReliabilityContext::GetWebUIData() const {
   DictionaryValue* context_value = new DictionaryValue();
 
@@ skipping 133 matching lines @@
 
   beacons_.pop_front();
 
   // If that just removed a beacon counted in uploading_beacons_size_, decrement
   // that.
   if (uploading_beacons_size_ > 0)
     --uploading_beacons_size_;
 }
 
 }  // namespace domain_reliability