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

Unified Diff: chrome/browser/stack_sampling_configuration.h

Issue 2375563002: Stack sampling profiler: move configuration to chrome/common (Closed)
Patch Set: remove closure 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/stack_sampling_configuration.h
diff --git a/chrome/browser/stack_sampling_configuration.h b/chrome/browser/stack_sampling_configuration.h
deleted file mode 100644
index 9a52b037b7e4769c7d4980a9603465f009b1992f..0000000000000000000000000000000000000000
--- a/chrome/browser/stack_sampling_configuration.h
+++ /dev/null
@@ -1,44 +0,0 @@
-// Copyright 2015 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.
-
-#ifndef CHROME_BROWSER_STACK_SAMPLING_CONFIGURATION_H_
-#define CHROME_BROWSER_STACK_SAMPLING_CONFIGURATION_H_
-
-#include "base/macros.h"
-#include "base/profiler/stack_sampling_profiler.h"
-
-// Chooses a configuration for the stack sampling profiler for browser process
-// startup. This must live outside of ChromeBrowserMainParts so it can be
-// friended by ChromeMetricsServiceAccessor.
-class StackSamplingConfiguration {
- public:
- StackSamplingConfiguration();
-
- // Get the stack sampling params to use for this session.
- base::StackSamplingProfiler::SamplingParams GetSamplingParams() const;
-
- // Returns true if the profiler should be started at all.
- bool IsProfilerEnabled() const;
-
- // Register the chosen configuration as a synthetic field trial.
- void RegisterSyntheticFieldTrial() const;
-
- private:
- enum ProfileConfiguration {
- PROFILE_DISABLED,
- PROFILE_CONTROL,
- PROFILE_NO_SAMPLES, // Run the profiler thread, but don't collect profiles.
- PROFILE_5HZ,
- PROFILE_10HZ,
- PROFILE_100HZ
- };
-
- static ProfileConfiguration GenerateConfiguration();
-
- const ProfileConfiguration configuration_;
-
- DISALLOW_COPY_AND_ASSIGN(StackSamplingConfiguration);
-};
-
-#endif // CHROME_BROWSER_STACK_SAMPLING_CONFIGURATION_H_

Powered by Google App Engine
This is Rietveld 408576698