Slow Reports - Embed Metadata in Traces

content/public/browser/tracing_delegate.h

Index: content/public/browser/tracing_delegate.h
diff --git a/content/public/browser/tracing_delegate.h b/content/public/browser/tracing_delegate.h
index b7cfa2d4fb0e4ec3af16ec47adf63723599ad63c..1c762309bad1ddcc632d5b6bdb20865e87772fe0 100644
--- a/content/public/browser/tracing_delegate.h
+++ b/content/public/browser/tracing_delegate.h
@@ -5,6 +5,8 @@
 #ifndef CONTENT_PUBLIC_BROWSER_TRACING_DELEGATE_H_
 #define CONTENT_PUBLIC_BROWSER_TRACING_DELEGATE_H_
 
+#include <string>
+
 #include "base/memory/scoped_ptr.h"
 #include "content/common/content_export.h"
 
@@ -30,6 +32,8 @@ class CONTENT_EXPORT TracingDelegate {
   virtual scoped_ptr<TraceUploader> GetTraceUploader(
       net::URLRequestContextGetter* request_context) = 0;
 
+  virtual bool GetChromeVersion(std::string*) = 0;

[oystein (OOO til 10th of July), 2015/06/12 18:14:17]

We have this in the content client already, I'm pretty sure? GetProduct(). Also,
"Chrome" shouldn't be mentioned in content/.

[shatch, 2015/06/12 19:05:54]

Sweet, tried to find where I could get it in content.

+
   // This can be used to veto a particular background tracing scenario.
   virtual bool IsAllowedToBeginBackgroundScenario(
       const BackgroundTracingConfig& config,