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

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 5fab907c1022e8cfecbcddc1029357abf15f92d7..3eb8c136b28241ddb062b1fd04b418130e28c9b0 100644
--- a/content/public/browser/tracing_delegate.h
+++ b/content/public/browser/tracing_delegate.h
@@ -7,11 +7,16 @@
 
 #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
@@ -23,6 +28,15 @@ class 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