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

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

Issue 2257533005: Change logging of about:flags from actions to a histogram. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix typo in header. Created 4 years, 3 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
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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_browser_main_extra_parts_metrics.h" 5 #include "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 301 matching lines...) Expand 10 before | Expand all | Expand 10 after
312 ChromeBrowserMainExtraPartsMetrics::~ChromeBrowserMainExtraPartsMetrics() { 312 ChromeBrowserMainExtraPartsMetrics::~ChromeBrowserMainExtraPartsMetrics() {
313 if (is_screen_observer_) 313 if (is_screen_observer_)
314 display::Screen::GetScreen()->RemoveObserver(this); 314 display::Screen::GetScreen()->RemoveObserver(this);
315 } 315 }
316 316
317 void ChromeBrowserMainExtraPartsMetrics::PreProfileInit() { 317 void ChromeBrowserMainExtraPartsMetrics::PreProfileInit() {
318 RecordMicroArchitectureStats(); 318 RecordMicroArchitectureStats();
319 } 319 }
320 320
321 void ChromeBrowserMainExtraPartsMetrics::PreBrowserStart() { 321 void ChromeBrowserMainExtraPartsMetrics::PreBrowserStart() {
322 flags_ui::PrefServiceFlagsStorage flags_storage_( 322 flags_ui::PrefServiceFlagsStorage flags_storage(
323 g_browser_process->local_state()); 323 g_browser_process->local_state());
324 about_flags::RecordUMAStatistics(&flags_storage_); 324 about_flags::RecordUMAStatistics(&flags_storage);
325 } 325 }
326 326
327 void ChromeBrowserMainExtraPartsMetrics::PostBrowserStart() { 327 void ChromeBrowserMainExtraPartsMetrics::PostBrowserStart() {
328 RecordLinuxGlibcVersion(); 328 RecordLinuxGlibcVersion();
329 #if defined(USE_X11) && !defined(OS_CHROMEOS) 329 #if defined(USE_X11) && !defined(OS_CHROMEOS)
330 UMA_HISTOGRAM_ENUMERATION("Linux.WindowManager", 330 UMA_HISTOGRAM_ENUMERATION("Linux.WindowManager",
331 GetLinuxWindowManager(), 331 GetLinuxWindowManager(),
332 UMA_LINUX_WINDOW_MANAGER_COUNT); 332 UMA_LINUX_WINDOW_MANAGER_COUNT);
333 #endif 333 #endif
334 334
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
394 } 394 }
395 } 395 }
396 396
397 namespace chrome { 397 namespace chrome {
398 398
399 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts) { 399 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts) {
400 main_parts->AddParts(new ChromeBrowserMainExtraPartsMetrics()); 400 main_parts->AddParts(new ChromeBrowserMainExtraPartsMetrics());
401 } 401 }
402 402
403 } // namespace chrome 403 } // namespace chrome
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698