Moved the TraceUploader to chrome/ and abstracted it for alternative new upload destinations.

chrome/browser/tracing/chrome_tracing_delegate.h

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
new file mode 100644
index 0000000000000000000000000000000000000000..0cdbf7280d991f6f0b991d3aa76e07badfff6b37
--- /dev/null
+++ b/chrome/browser/tracing/chrome_tracing_delegate.h
@@ -0,0 +1,16 @@
+// 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_TRACING_CHROME_TRACING_DELEGATE_H_
+#define CHROME_BROWSER_TRACING_CHROME_TRACING_DELEGATE_H_
+
+#include "content/public/browser/tracing_delegate.h"
+
+class ChromeTracingDelegate : public content::TracingDelegate {
+ public:
+  content::TraceUploader* GetTraceUploader(

[Lei Zhang, 2015/02/10 22:28:16]

Is it possible to change the interface to return a scoped_ptr so it's obvious
that the callers takes ownership of the TraceUploader?

[oystein (OOO til 10th of July), 2015/02/10 23:40:15]

Done. Looks like the original code here was leaking this object each time.

+      net::URLRequestContextGetter* request_context) override;
+};
+
+#endif  // CHROME_BROWSER_TRACING_CHROME_TRACING_DELEGATE_H_