Thread-local trace-event buffers

base/debug/trace_event_android.cc

 // Copyright (c) 2012 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 "base/debug/trace_event_impl.h"
 
 #include <fcntl.h>
 
 #include "base/debug/trace_event.h"
 #include "base/format_macros.h"
@@ skipping 68 matching lines @@
   AutoLock lock(lock_);
   if (g_atrace_fd != -1) {
     close(g_atrace_fd);
     g_atrace_fd = -1;
     UpdateCategoryGroupEnabledFlags();
   }
 }
 
 void TraceLog::SendToATrace(
     char phase,
-    const char* category_group,
+    const unsigned char* category_group_enabled,
     const char* name,
     unsigned long long id,
     int num_args,
     const char** arg_names,
     const unsigned char* arg_types,
     const unsigned long long* arg_values,
     scoped_ptr<ConvertableToTraceFormat> convertable_values[],
     unsigned char flags) {
   if (g_atrace_fd == -1)
     return;
 
+  const char* category_group = GetCategoryGroupName(category_group_enabled);

[dsinclair, 2013/08/13 21:04:42]

This seems unrelated, move to its own CL?

[Xianzhu, 2013/08/14 21:28:33]

Done.

   switch (phase) {
     case TRACE_EVENT_PHASE_BEGIN:
       WriteEvent('B', category_group, name, id,
                  num_args, arg_names, arg_types, arg_values, convertable_values,
                  flags);
       break;
 
     case TRACE_EVENT_PHASE_END:
       // Though a single 'E' is enough, here append pid, name and
       // category_group etc. So that unpaired events can be found easily.
@@ skipping 38 matching lines @@
   const char* category_group = GetCategoryGroupName(category_group_enabled);
   if (strncmp(category_group, TRACE_DISABLED_BY_DEFAULT(""),
               strlen(TRACE_DISABLED_BY_DEFAULT(""))) == 0)
     return;
 
   *category_group_enabled |= ATRACE_ENABLED;
 }
 
 }  // namespace debug
 }  // namespace base