Reland of Background tracing: Added config option for repeated trigger behavior (patchset #1 id:1 o…

content/browser/tracing/background_tracing_manager_impl.cc

 // 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.
 
 #include "content/browser/tracing/background_tracing_manager_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/json/json_writer.h"
@@ skipping 311 matching lines @@
   if (config_->tracing_mode() == BackgroundTracingConfigImpl::REACTIVE) {
     // In reactive mode, a trigger starts tracing, or finalizes tracing
     // immediately if it's already running.
     RecordBackgroundTracingMetric(REACTIVE_TRIGGERED);
 
     if (!is_tracing_) {
       // It was not already tracing, start a new trace.
       StartTracing(triggered_rule->category_preset(),
                    base::trace_event::RECORD_UNTIL_FULL);
     } else {
-      // Reactive configs that trigger again while tracing should just
+      // Some reactive configs that trigger again while tracing should just
       // end right away (to not capture multiple navigations, for example).
-      trace_delay = -1;
+      // For others we just want to ignore the repeated trigger.
+      if (triggered_rule->stop_tracing_on_repeated_reactive()) {
+        trace_delay = -1;
+      } else {
+        if (!callback.is_null())
+          callback.Run(false);
+        return;
+      }
     }
   } else {
     // In preemptive mode, a trigger starts finalizing a trace if one is
     // running and we're not got a finalization timer running,
     // otherwise we do nothing.
     if (!is_tracing_ || is_gathering_ || tracing_timer_) {
       if (!callback.is_null())
         callback.Run(false);
       return;
     }
@@ skipping 214 matching lines @@
     case BackgroundTracingConfigImpl::CategoryPreset::BLINK_STYLE:
       return "blink_style";
     case BackgroundTracingConfigImpl::CategoryPreset::CATEGORY_PRESET_UNSET:
       NOTREACHED();
   }
   NOTREACHED();
   return "";
 }
 
 }  // namspace content