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

Side by Side Diff: chrome/browser/metrics/chrome_metrics_service_client.cc

Issue 464463003: Adding logging of offstore extensions state to user metrics. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/metrics/chrome_metrics_service_client.h" 5 #include "chrome/browser/metrics/chrome_metrics_service_client.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 263 matching lines...) Expand 10 before | Expand all | Expand 10 after
274 #else 274 #else
275 NOTREACHED(); 275 NOTREACHED();
276 #endif // defined(ENABLE_PLUGINS) 276 #endif // defined(ENABLE_PLUGINS)
277 } 277 }
278 278
279 void ChromeMetricsServiceClient::Initialize() { 279 void ChromeMetricsServiceClient::Initialize() {
280 metrics_service_.reset(new MetricsService( 280 metrics_service_.reset(new MetricsService(
281 metrics_state_manager_, this, g_browser_process->local_state())); 281 metrics_state_manager_, this, g_browser_process->local_state()));
282 282
283 // Register metrics providers. 283 // Register metrics providers.
284 #if defined(ENABLE_EXTENSIONS)
Alexei Svitkine (slow) 2014/08/13 20:30:30 Can you ifdef the header inclusion too?
jwd 2014/08/13 20:36:32 Done.
284 metrics_service_->RegisterMetricsProvider( 285 metrics_service_->RegisterMetricsProvider(
285 scoped_ptr<metrics::MetricsProvider>( 286 scoped_ptr<metrics::MetricsProvider>(
286 new ExtensionsMetricsProvider(metrics_state_manager_))); 287 new ExtensionsMetricsProvider(metrics_state_manager_)));
288 #endif
287 metrics_service_->RegisterMetricsProvider( 289 metrics_service_->RegisterMetricsProvider(
288 scoped_ptr<metrics::MetricsProvider>(new NetworkMetricsProvider)); 290 scoped_ptr<metrics::MetricsProvider>(new NetworkMetricsProvider));
289 metrics_service_->RegisterMetricsProvider( 291 metrics_service_->RegisterMetricsProvider(
290 scoped_ptr<metrics::MetricsProvider>(new OmniboxMetricsProvider)); 292 scoped_ptr<metrics::MetricsProvider>(new OmniboxMetricsProvider));
291 metrics_service_->RegisterMetricsProvider( 293 metrics_service_->RegisterMetricsProvider(
292 scoped_ptr<metrics::MetricsProvider>(new ChromeStabilityMetricsProvider)); 294 scoped_ptr<metrics::MetricsProvider>(new ChromeStabilityMetricsProvider));
293 metrics_service_->RegisterMetricsProvider( 295 metrics_service_->RegisterMetricsProvider(
294 scoped_ptr<metrics::MetricsProvider>(new GPUMetricsProvider())); 296 scoped_ptr<metrics::MetricsProvider>(new GPUMetricsProvider()));
295 profiler_metrics_provider_ = new ProfilerMetricsProvider; 297 profiler_metrics_provider_ = new ProfilerMetricsProvider;
296 metrics_service_->RegisterMetricsProvider( 298 metrics_service_->RegisterMetricsProvider(
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
539 // Capture the histogram samples. 541 // Capture the histogram samples.
540 if (dumps_with_crash != 0) 542 if (dumps_with_crash != 0)
541 UMA_HISTOGRAM_COUNTS("Chrome.BrowserDumpsWithCrash", dumps_with_crash); 543 UMA_HISTOGRAM_COUNTS("Chrome.BrowserDumpsWithCrash", dumps_with_crash);
542 if (dumps_with_no_crash != 0) 544 if (dumps_with_no_crash != 0)
543 UMA_HISTOGRAM_COUNTS("Chrome.BrowserDumpsWithNoCrash", dumps_with_no_crash); 545 UMA_HISTOGRAM_COUNTS("Chrome.BrowserDumpsWithNoCrash", dumps_with_no_crash);
544 int total_dumps = dumps_with_crash + dumps_with_no_crash; 546 int total_dumps = dumps_with_crash + dumps_with_no_crash;
545 if (total_dumps != 0) 547 if (total_dumps != 0)
546 UMA_HISTOGRAM_COUNTS("Chrome.BrowserCrashDumpAttempts", total_dumps); 548 UMA_HISTOGRAM_COUNTS("Chrome.BrowserCrashDumpAttempts", total_dumps);
547 } 549 }
548 #endif // defined(OS_WIN) 550 #endif // defined(OS_WIN)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698