Enable overlapping, usage counted SetEnabled()/SetDisabled() for TraceLog

base/debug/trace_event_impl.cc

Index: base/debug/trace_event_impl.cc
diff --git a/base/debug/trace_event_impl.cc b/base/debug/trace_event_impl.cc
index 07679d1131250fb101401191d02434dc6e59e8b8..1126b6eeaf85cdafd9a0e560b139f252154f53a1 100644
--- a/base/debug/trace_event_impl.cc
+++ b/base/debug/trace_event_impl.cc
@@ -346,7 +346,7 @@ TraceLog* TraceLog::GetInstance() {
 }
 
 TraceLog::TraceLog()
-    : enabled_(false),
+    : enable_count_(0),
       dispatching_to_observer_list_(false),
       watch_category_(NULL) {
   // Trace is enabled or disabled on one thread while other threads are
@@ -444,7 +444,7 @@ const unsigned char* TraceLog::GetCategoryEnabledInternal(const char* name) {
       ANNOTATE_LEAKING_OBJECT_PTR(new_name);
       g_categories[new_index] = new_name;
       DCHECK(!g_category_enabled[new_index]);
-      if (enabled_) {
+      if (enable_count_) {
         // Note that if both included and excluded_categories are empty, the
         // else clause below excludes nothing, thereby enabling this category.
         if (!included_categories_.empty()) {
@@ -477,7 +477,7 @@ void TraceLog::GetKnownCategories(std::vector<std::string>* categories) {
 void TraceLog::SetEnabled(const std::vector<std::string>& included_categories,
                           const std::vector<std::string>& excluded_categories) {
   AutoLock lock(lock_);
-  if (enabled_)
+  if (enable_count_++ > 0)
     return;
 
   if (dispatching_to_observer_list_) {
@@ -492,7 +492,6 @@ void TraceLog::SetEnabled(const std::vector<std::string>& included_categories,
   dispatching_to_observer_list_ = false;
 
   logged_events_.reserve(1024);
-  enabled_ = true;
   included_categories_ = included_categories;
   excluded_categories_ = excluded_categories;
   // Note that if both included and excluded_categories are empty, the else
@@ -528,7 +527,7 @@ void TraceLog::GetEnabledTraceCategories(
     std::vector<std::string>* included_out,
     std::vector<std::string>* excluded_out) {
   AutoLock lock(lock_);
-  if (enabled_) {
+  if (enable_count_) {
     *included_out = included_categories_;
     *excluded_out = excluded_categories_;
   }
@@ -536,7 +535,8 @@ void TraceLog::GetEnabledTraceCategories(
 
 void TraceLog::SetDisabled() {
   AutoLock lock(lock_);
-  if (!enabled_)
+  DCHECK(enable_count_ > 0);
+  if (--enable_count_ != 0)
     return;
 
   if (dispatching_to_observer_list_) {
@@ -550,7 +550,6 @@ void TraceLog::SetDisabled() {
                     OnTraceLogWillDisable());
   dispatching_to_observer_list_ = false;
 
-  enabled_ = false;
   included_categories_.clear();
   excluded_categories_.clear();
   watch_category_ = NULL;