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

Unified Diff: chrome/browser/tracing/chrome_tracing_delegate.h

Issue 1164023002: Slow Reports: Set a minimum time in between reports, and check for OTR sessions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: content_browsertest fixes Created 5 years, 6 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/tracing/chrome_tracing_delegate.h
diff --git a/chrome/browser/tracing/chrome_tracing_delegate.h b/chrome/browser/tracing/chrome_tracing_delegate.h
index b1c97398bd3cf49668a73e6876b4515000b71856..0d73482b45fcfe6965fe10f6ecf0b130d321cfab 100644
--- a/chrome/browser/tracing/chrome_tracing_delegate.h
+++ b/chrome/browser/tracing/chrome_tracing_delegate.h
@@ -5,12 +5,35 @@
#ifndef CHROME_BROWSER_TRACING_CHROME_TRACING_DELEGATE_H_
#define CHROME_BROWSER_TRACING_CHROME_TRACING_DELEGATE_H_
+#include "chrome/browser/ui/browser_list_observer.h"
#include "content/public/browser/tracing_delegate.h"
-class ChromeTracingDelegate : public content::TracingDelegate {
+class PrefRegistrySimple;
+
+void RegisterTracingPrefs(PrefRegistrySimple* registry);
gab 2015/06/05 14:02:41 Make this a static call of ChromeTracingDelegate.
oystein (OOO til 10th of July) 2015/06/05 21:14:42 Done.
+
+class ChromeTracingDelegate : public content::TracingDelegate,
+ public chrome::BrowserListObserver {
public:
+ ChromeTracingDelegate();
+ ~ChromeTracingDelegate() override;
+
scoped_ptr<content::TraceUploader> GetTraceUploader(
net::URLRequestContextGetter* request_context) override;
+
+ bool IsAllowedToBeginBackgroundScenario(
+ const content::BackgroundTracingConfig& config,
+ bool requires_anonymized_data) override;
+
+ bool IsAllowedToEndBackgroundScenario(
+ const content::BackgroundTracingConfig& config,
+ bool requires_anonymized_data) override;
+
+ private:
+ // chrome::BrowserListObserver implementation.
+ void OnBrowserAdded(Browser* browser) override;
+
+ bool incognito_launched_;
};
#endif // CHROME_BROWSER_TRACING_CHROME_TRACING_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698