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

Unified Diff: chrome/browser/feedback/tracing_manager.h

Issue 67683003: Remove TraceController (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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/feedback/tracing_manager.h
diff --git a/chrome/browser/feedback/tracing_manager.h b/chrome/browser/feedback/tracing_manager.h
index c70206ae603546e6972412788bfa64ba46678814..5fd5588326d3b75e2de1744558ad5c74eb48bb54 100644
--- a/chrome/browser/feedback/tracing_manager.h
+++ b/chrome/browser/feedback/tracing_manager.h
@@ -12,13 +12,18 @@
#include "base/callback.h"
#include "base/memory/scoped_ptr.h"
#include "base/memory/weak_ptr.h"
-#include "content/public/browser/trace_subscriber.h"
+namespace base {
+
+class RefCountedString;
+class FilePath;
+
+}
// Callback used for getting the output of a trace.
typedef base::Callback<void(scoped_refptr<base::RefCountedString> trace_data)>
TraceDataCallback;
-// This class is used to manage performance meterics that can be attached to
+// This class is used to manage performance metrics that can be attached to
// feedback reports. This class is a Singleton that is owned by the preference
// system. It should only be created when it is enabled, and should only be
// accessed elsewhere via Get().
@@ -28,7 +33,7 @@ typedef base::Callback<void(scoped_refptr<base::RefCountedString> trace_data)>
// version of the performance data. That data can then be requested via
// GetTraceData(). When the data is no longer needed, it should be discarded
// via DiscardTraceData().
-class TracingManager : public content::TraceSubscriber {
+class TracingManager {
public:
virtual ~TracingManager();
@@ -50,17 +55,10 @@ class TracingManager : public content::TraceSubscriber {
void DiscardTraceData(int id);
private:
- void StartTracing();
-
- // content::TraceSubscriber overrides
- virtual void OnEndTracingComplete() OVERRIDE;
- virtual void OnTraceDataCollected(
- const scoped_refptr<base::RefCountedString>& trace_fragment) OVERRIDE;
-
TracingManager();
- // Data being collected from the current trace.
- std::string data_;
+ void StartTracing();
+ void OnTraceDataCollected(const base::FilePath& path);
// ID of the trace that is being collected.
int current_trace_id_;

Powered by Google App Engine
This is Rietveld 408576698