Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(626)

Unified Diff: net/disk_cache/blockfile/stats_histogram.cc

Issue 295913014: Disk cache: Remove stats_histogram. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix typo Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/disk_cache/blockfile/stats_histogram.h ('k') | net/net.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/disk_cache/blockfile/stats_histogram.cc
diff --git a/net/disk_cache/blockfile/stats_histogram.cc b/net/disk_cache/blockfile/stats_histogram.cc
deleted file mode 100644
index a95eb46f4373e5334a8a274bd50b2dffac482250..0000000000000000000000000000000000000000
--- a/net/disk_cache/blockfile/stats_histogram.cc
+++ /dev/null
@@ -1,94 +0,0 @@
-// 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 "net/disk_cache/blockfile/stats_histogram.h"
-
-#include "base/debug/leak_annotations.h"
-#include "base/logging.h"
-#include "base/metrics/bucket_ranges.h"
-#include "base/metrics/histogram_base.h"
-#include "base/metrics/sample_vector.h"
-#include "base/metrics/statistics_recorder.h"
-#include "net/disk_cache/blockfile/stats.h"
-
-namespace disk_cache {
-
-using base::BucketRanges;
-using base::Histogram;
-using base::HistogramSamples;
-using base::SampleVector;
-using base::StatisticsRecorder;
-
-StatsHistogram::StatsHistogram(const std::string& name,
- Sample minimum,
- Sample maximum,
- const BucketRanges* ranges,
- const Stats* stats)
- : Histogram(name, minimum, maximum, ranges),
- stats_(stats) {}
-
-StatsHistogram::~StatsHistogram() {}
-
-// static
-void StatsHistogram::InitializeBucketRanges(const Stats* stats,
- BucketRanges* ranges) {
- for (size_t i = 0; i < ranges->size(); ++i) {
- ranges->set_range(i, stats->GetBucketRange(i));
- }
- ranges->ResetChecksum();
-}
-
-StatsHistogram* StatsHistogram::FactoryGet(const std::string& name,
- const Stats* stats) {
- Sample minimum = 1;
- Sample maximum = disk_cache::Stats::kDataSizesLength - 1;
- size_t bucket_count = disk_cache::Stats::kDataSizesLength;
- HistogramBase* histogram = StatisticsRecorder::FindHistogram(name);
- if (!histogram) {
- DCHECK(stats);
-
- // To avoid racy destruction at shutdown, the following will be leaked.
- BucketRanges* ranges = new BucketRanges(bucket_count + 1);
- InitializeBucketRanges(stats, ranges);
- const BucketRanges* registered_ranges =
- StatisticsRecorder::RegisterOrDeleteDuplicateRanges(ranges);
-
- // To avoid racy destruction at shutdown, the following will be leaked.
- StatsHistogram* stats_histogram =
- new StatsHistogram(name, minimum, maximum, registered_ranges, stats);
- stats_histogram->SetFlags(kUmaTargetedHistogramFlag);
- histogram = StatisticsRecorder::RegisterOrDeleteDuplicate(stats_histogram);
- }
-
- DCHECK(base::HISTOGRAM == histogram->GetHistogramType());
- DCHECK(histogram->HasConstructionArguments(minimum, maximum, bucket_count));
-
- // We're preparing for an otherwise unsafe upcast by ensuring we have the
- // proper class type.
- StatsHistogram* return_histogram = static_cast<StatsHistogram*>(histogram);
- return return_histogram;
-}
-
-void StatsHistogram::Disable() {
- stats_ = NULL;
-}
-
-scoped_ptr<HistogramSamples> StatsHistogram::SnapshotSamples() const {
- scoped_ptr<SampleVector> samples(new SampleVector(bucket_ranges()));
- if (stats_)
- stats_->Snapshot(samples.get());
-
- // Only report UMA data once.
- StatsHistogram* mutable_me = const_cast<StatsHistogram*>(this);
- mutable_me->ClearFlags(kUmaTargetedHistogramFlag);
-
- return samples.PassAs<HistogramSamples>();
-}
-
-int StatsHistogram::FindCorruption(const HistogramSamples& samples) const {
- // This class won't monitor inconsistencies.
- return HistogramBase::NO_INCONSISTENCIES;
-}
-
-} // namespace disk_cache
« no previous file with comments | « net/disk_cache/blockfile/stats_histogram.h ('k') | net/net.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698