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

Unified Diff: chrome/browser/extensions/extension_metrics_module.cc

Issue 10696145: Moving metrics to api/ . (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Putting back "using" and updating WATCHLISTS Created 8 years, 5 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 | « chrome/browser/extensions/extension_metrics_module.h ('k') | chrome/chrome_browser_extensions.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extension_metrics_module.cc
diff --git a/chrome/browser/extensions/extension_metrics_module.cc b/chrome/browser/extensions/extension_metrics_module.cc
deleted file mode 100644
index 0c63a7aee1ab21c182ef9a2a8e31b8e5692539df..0000000000000000000000000000000000000000
--- a/chrome/browser/extensions/extension_metrics_module.cc
+++ /dev/null
@@ -1,148 +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 "chrome/browser/extensions/extension_metrics_module.h"
-
-#include <algorithm>
-
-#include "base/metrics/histogram.h"
-#include "chrome/common/extensions/extension.h"
-#include "content/public/browser/user_metrics.h"
-
-using base::Histogram;
-using base::LinearHistogram;
-using content::UserMetricsAction;
-
-const size_t kMaxBuckets = 10000; // We don't ever want more than these many
- // buckets; there is no real need for them
- // and would cause crazy memory usage
-
-bool MetricsRecordUserActionFunction::RunImpl() {
- std::string name;
- EXTENSION_FUNCTION_VALIDATE(args_->GetString(0, &name));
-
- content::RecordComputedAction(name);
- return true;
-}
-
-bool MetricsHistogramHelperFunction::GetNameAndSample(std::string* name,
- int* sample) {
- EXTENSION_FUNCTION_VALIDATE(args_->GetString(0, name));
- EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(1, sample));
- return true;
-}
-
-bool MetricsHistogramHelperFunction::RecordValue(const std::string& name,
- Histogram::ClassType type,
- int min,
- int max,
- size_t buckets,
- int sample) {
- // Make sure toxic values don't get to internal code.
- // Fix for maximums
- min = std::min(min, INT_MAX - 3);
- max = std::min(max, INT_MAX - 3);
- buckets = std::min(buckets, kMaxBuckets);
- // Fix for minimums.
- min = std::max(min, 1);
- max = std::max(max, min + 1);
- buckets = std::max(buckets, static_cast<size_t>(3));
- // Trim buckets down to a maximum of the given range + over/underflow buckets
- if (buckets > static_cast<size_t>(max - min + 2))
- buckets = max - min + 2;
-
- Histogram* counter;
- if (type == Histogram::LINEAR_HISTOGRAM) {
- counter = LinearHistogram::FactoryGet(name,
- min,
- max,
- buckets,
- Histogram::kUmaTargetedHistogramFlag);
- } else {
- counter = Histogram::FactoryGet(name,
- min,
- max,
- buckets,
- Histogram::kUmaTargetedHistogramFlag);
- }
-
- counter->Add(sample);
- return true;
-}
-
-bool MetricsRecordValueFunction::RunImpl() {
- int sample;
- EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(1, &sample));
-
- // Get the histogram parameters from the metric type object.
- DictionaryValue* metric_type;
- EXTENSION_FUNCTION_VALIDATE(args_->GetDictionary(0, &metric_type));
-
- std::string name;
- std::string type;
- int min;
- int max;
- int buckets;
- EXTENSION_FUNCTION_VALIDATE(metric_type->GetString("metricName", &name));
- EXTENSION_FUNCTION_VALIDATE(metric_type->GetString("type", &type));
- EXTENSION_FUNCTION_VALIDATE(metric_type->GetInteger("min", &min));
- EXTENSION_FUNCTION_VALIDATE(metric_type->GetInteger("max", &max));
- EXTENSION_FUNCTION_VALIDATE(metric_type->GetInteger("buckets", &buckets));
-
- Histogram::ClassType histogram_type(type == "histogram-linear" ?
- Histogram::LINEAR_HISTOGRAM : Histogram::HISTOGRAM);
- return RecordValue(name, histogram_type, min, max, buckets, sample);
-}
-
-bool MetricsRecordPercentageFunction::RunImpl() {
- std::string name;
- int sample;
- EXTENSION_FUNCTION_VALIDATE(GetNameAndSample(&name, &sample));
- return RecordValue(name, Histogram::LINEAR_HISTOGRAM, 1, 101, 102, sample);
-}
-
-bool MetricsRecordCountFunction::RunImpl() {
- std::string name;
- int sample;
- EXTENSION_FUNCTION_VALIDATE(GetNameAndSample(&name, &sample));
- return RecordValue(name, Histogram::HISTOGRAM, 1, 1000000, 50, sample);
-}
-
-bool MetricsRecordSmallCountFunction::RunImpl() {
- std::string name;
- int sample;
- EXTENSION_FUNCTION_VALIDATE(GetNameAndSample(&name, &sample));
- return RecordValue(name, Histogram::HISTOGRAM, 1, 100, 50, sample);
-}
-
-bool MetricsRecordMediumCountFunction::RunImpl() {
- std::string name;
- int sample;
- EXTENSION_FUNCTION_VALIDATE(GetNameAndSample(&name, &sample));
- return RecordValue(name, Histogram::HISTOGRAM, 1, 10000, 50, sample);
-}
-
-bool MetricsRecordTimeFunction::RunImpl() {
- std::string name;
- int sample;
- EXTENSION_FUNCTION_VALIDATE(GetNameAndSample(&name, &sample));
- static const int kTenSecMs = 10 * 1000;
- return RecordValue(name, Histogram::HISTOGRAM, 1, kTenSecMs, 50, sample);
-}
-
-bool MetricsRecordMediumTimeFunction::RunImpl() {
- std::string name;
- int sample;
- EXTENSION_FUNCTION_VALIDATE(GetNameAndSample(&name, &sample));
- static const int kThreeMinMs = 3 * 60 * 1000;
- return RecordValue(name, Histogram::HISTOGRAM, 1, kThreeMinMs, 50, sample);
-}
-
-bool MetricsRecordLongTimeFunction::RunImpl() {
- std::string name;
- int sample;
- EXTENSION_FUNCTION_VALIDATE(GetNameAndSample(&name, &sample));
- static const int kOneHourMs = 60 * 60 * 1000;
- return RecordValue(name, Histogram::HISTOGRAM, 1, kOneHourMs, 50, sample);
-}
« no previous file with comments | « chrome/browser/extensions/extension_metrics_module.h ('k') | chrome/chrome_browser_extensions.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698