Display histograms from subprocesses in chrome://histograms

base/metrics/histogram.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.
 
 // Histogram is an object that aggregates statistics, and can summarize them in
 // various forms, including ASCII graphical, HTML, and numerically (as a
 // vector of numbers corresponding to each of the aggregating buckets).
 // See header file for details and examples.
 
 #include "base/metrics/histogram.h"
@@ skipping 444 matching lines @@
 
 void Histogram::AddSamples(const HistogramSamples& samples) {
   samples_->Add(samples);
 }
 
 bool Histogram::AddSamplesFromPickle(PickleIterator* iter) {
   return samples_->AddFromPickle(iter);
 }
 
 // The following methods provide a graphical histogram display.
-void Histogram::WriteHTMLGraph(std::string* output) const {
+void Histogram::WriteHTMLGraph(const HistogramSamples* snapshot,
+                               std::string* output) const {
   // TBD(jar) Write a nice HTML bar chart, with divs an mouse-overs etc.
   output->append("<PRE>");
-  WriteAsciiImpl(true, "<br>", output);
+  WriteAsciiImpl(true, "<br>", static_cast<const SampleVector*>(snapshot),

[Ilya Sherman, 2016/04/13 00:29:32]

Hmm, I'm not super happy about the static_cast here.  Why is it needed?  That
is, could either this method accept a SampleVector, or the WriteAsciiImpl method
accept a HistogramSamples param?

[bcwhite, 2016/04/13 11:58:18]

This is a virtual function off of HistogramBase so has to use the
HistogramSamples base class there (or it won't work with SparseHistogram).

I originally tried passing HistogramSamples all the way but WriteAsciiImpl needs
to call methods defined in (and only applicable to) SampleVector.

I could pass HistogramSamples to WriteAsciiImpl and do the static_cast in there
but that made for slightly more complicated code.  It seemed more clear to do
the cast in the overridden method.

+                 output);
   output->append("</PRE>");
 }
 
-void Histogram::WriteAscii(std::string* output) const {
-  WriteAsciiImpl(true, "\n", output);
+void Histogram::WriteAscii(const HistogramSamples* snapshot,
+                           std::string* output) const {
+  WriteAsciiImpl(true, "\n", static_cast<const SampleVector*>(snapshot),
+                 output);
 }
 
 bool Histogram::SerializeInfoImpl(Pickle* pickle) const {
   DCHECK(bucket_ranges()->HasValidChecksum());
   return pickle->WriteString(histogram_name()) &&
       pickle->WriteInt(flags()) &&
       pickle->WriteInt(declared_min()) &&
       pickle->WriteInt(declared_max()) &&
       pickle->WriteUInt32(bucket_count()) &&
       pickle->WriteUInt32(bucket_ranges()->checksum());
@@ skipping 87 matching lines @@
 
 std::unique_ptr<SampleVector> Histogram::SnapshotSampleVector() const {
   std::unique_ptr<SampleVector> samples(
       new SampleVector(samples_->id(), bucket_ranges()));
   samples->Add(*samples_);
   return samples;
 }
 
 void Histogram::WriteAsciiImpl(bool graph_it,
                                const std::string& newline,
+                               const SampleVector* snapshot,
                                std::string* output) const {
   // Get local (stack) copies of all effectively volatile class data so that we
   // are consistent across our output activities.

[Ilya Sherman, 2016/04/13 00:29:32]

nit: Mebbe move this comment into the if-stmt?

[bcwhite, 2016/04/13 11:58:18]

Done.

-  std::unique_ptr<SampleVector> snapshot = SnapshotSampleVector();
+  std::unique_ptr<SampleVector> local_snapshot;
+  if (!snapshot) {
+    local_snapshot = SnapshotSampleVector();
+    snapshot = local_snapshot.get();
+  }
   Count sample_count = snapshot->TotalCount();
 
   WriteAsciiHeader(*snapshot, sample_count, output);
   output->append(newline);
 
   // Prepare to normalize graphical rendering of bucket contents.
   double max_size = 0;
   if (graph_it)
     max_size = GetPeakBucketSize(*snapshot);
 
@@ skipping 570 matching lines @@
     Sample sample = custom_ranges[i];
     if (sample < 0 || sample > HistogramBase::kSampleType_MAX - 1)
       return false;
     if (sample != 0)
       has_valid_range = true;
   }
   return has_valid_range;
 }
 
 }  // namespace base