 Chromium Code Reviews
 Chromium Code Reviews Issue 1165673002:
  [Startup Tracing] Hook up TraceConfig and remove CategoryFilter & TraceOptions  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1165673002:
  [Startup Tracing] Hook up TraceConfig and remove CategoryFilter & TraceOptions  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: base/trace_event/trace_event_android.cc | 
| diff --git a/base/trace_event/trace_event_android.cc b/base/trace_event/trace_event_android.cc | 
| index 91ed6e3cf08f5c140c3b68b458ea5dbaf6eaa84d..465649d44d5acb5b4633ee6af72cf365148130f5 100644 | 
| --- a/base/trace_event/trace_event_android.cc | 
| +++ b/base/trace_event/trace_event_android.cc | 
| @@ -97,9 +97,9 @@ void TraceLog::StartATrace() { | 
| 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 ju
 
Zhen Wang
2015/06/02 16:24:42
We do not use kDefaultCategoryFilterString any mor
 | 
| + trace_config.SetTraceRecordMode(RECORD_CONTINUOUSLY); | 
| + SetEnabled(trace_config, TraceLog::RECORDING_MODE); | 
| } | 
| void TraceLog::StopATrace() { |