Slow Reports: Set a minimum time in between reports, and check for OTR sessions

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 "base/memory/scoped_ptr.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 TracingDelegate {
  public:
   virtual ~TracingDelegate() {}
 
   // Provide trace uploading functionality; see trace_uploader.h.
   virtual scoped_ptr<TraceUploader> GetTraceUploader(
       net::URLRequestContextGetter* request_context) = 0;
+
+  // This can be used to veto a particular background tracing scenario.
+  virtual bool IsAllowedToBeginBackgroundScenario(
+      const BackgroundTracingConfig& config,
+      bool requires_anonymized_data) = 0;

[no sievers, 2015/06/04 22:55:34]

nit: Usually the interfaces to be implemented by the embedder have default
implementation stubs, though I don't know if it makes a lot of sense to *not*
implement any of these three functions here in this specific case

[oystein (OOO til 10th of July), 2015/06/05 21:14:42]

Fair point; I've made them default to false, as a safeguard.

+
+  virtual bool IsAllowedToEndBackgroundScenario(
+      const content::BackgroundTracingConfig& config,
+      bool requires_anonymized_data) = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_TRACING_DELEGATE_H_