Slow Reports - Embed Metadata in Traces

content/public/browser/tracing_delegate.h

 // 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 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"
 
 namespace base {
 class Time;
 }
 
 namespace net {
 class URLRequestContextGetter;
 }
 
 namespace content {
 struct BackgroundTracingConfig;
 class TraceUploader;
 
 // This can be implemented by the embedder to provide functionality for the
 // about://tracing WebUI.
 class CONTENT_EXPORT TracingDelegate {
  public:
   virtual ~TracingDelegate() {}
 
   // Provide trace uploading functionality; see trace_uploader.h.
   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,
       bool requires_anonymized_data);
 
   virtual bool IsAllowedToEndBackgroundScenario(
       const content::BackgroundTracingConfig& config,
       bool requires_anonymized_data);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_TRACING_DELEGATE_H_