[Startup Tracing] Hook up TraceConfig and remove CategoryFilter & TraceOptions

base/trace_event/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/trace_event/trace_event_impl.h"
 
 #include <fcntl.h>
 
 #include "base/format_macros.h"
 #include "base/logging.h"
@@ skipping 79 matching lines @@
 
 void TraceLog::StartATrace() {
   if (g_atrace_fd != -1)
     return;
 
   g_atrace_fd = open(kATraceMarkerFile, O_WRONLY);
   if (g_atrace_fd == -1) {
     PLOG(WARNING) << "Couldn't open " << kATraceMarkerFile;
     return;
   }
-  SetEnabled(CategoryFilter(CategoryFilter::kDefaultCategoryFilterString),
-             TraceLog::RECORDING_MODE,
-             TraceOptions(RECORD_CONTINUOUSLY));
+  TraceConfig trace_config;

[dsinclair, 2015/06/02 13:56:55]

Should we set kDefaultCategoryFilterString here just to make it explicit?

[Zhen Wang, 2015/06/02 16:24:42]

We do not use kDefaultCategoryFilterString any more because we now use
TraceConfig::InitializeDefault() to initialize the default values.

+  trace_config.SetTraceRecordMode(RECORD_CONTINUOUSLY);
+  SetEnabled(trace_config, TraceLog::RECORDING_MODE);
 }
 
 void TraceLog::StopATrace() {
   if (g_atrace_fd == -1)
     return;
 
   close(g_atrace_fd);
   g_atrace_fd = -1;
 
   // TraceLog::Flush() requires the current thread to have a message loop, but
@@ skipping 77 matching lines @@
   double now_in_seconds = (TraceTicks::Now() - TraceTicks()).InSecondsF();
   std::string marker = StringPrintf(
       "trace_event_clock_sync: parent_ts=%f\n", now_in_seconds);
   if (write(atrace_fd, marker.c_str(), marker.size()) == -1)
     PLOG(WARNING) << "Couldn't write to " << kATraceMarkerFile;
   close(atrace_fd);
 }
 
 }  // namespace trace_event
 }  // namespace base