Refactor trace_event_impl's SetEnabled to use TraceOptions. Propagate this through the whole stack.

base/debug/trace_event_impl_constants.cc

 // Copyright (c) 2013 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"
 
 namespace base {
 namespace debug {
 
 // Enable everything but debug and test categories by default.
 const char* CategoryFilter::kDefaultCategoryFilterString = "-*Debug,-*Test";
 
+// Constant used by TraceLog's internal implementation of trace_option.

[dsinclair, 2014/07/30 14:25:24]

nit: Constants

[nednguyen, 2014/07/30 16:51:13]

Done.

+const TraceLog::InternalTraceOptions TraceLog::kInternalNone = 0;
+const TraceLog::InternalTraceOptions TraceLog::kInternalRecordUntilFull =
+    1 << 0;
+const TraceLog::InternalTraceOptions TraceLog::kInternalRecordContinuously =
+    1 << 1;
+const TraceLog::InternalTraceOptions TraceLog::kInternalEnableSampling = 1
+                                                                          << 2;

[dsinclair, 2014/07/30 14:25:24]

This wrapping is nasty. What about something like:

const TraceLog::InternalTraceOptions
    TraceLog::kInternalNone = 0;
const TraceLog::InternalTraceOptions
    TraceLog::kInternalRecordUntilFull = 1 << 0;
const TraceLog::InternalTraceOptions
    TraceLog::kInternalRecordContinuously = 1 << 1;
const TraceLog::InternalTraceOptions
    TraceLog::kInternalEnableSampling = 1 << 2;
const TraceLog::InternalTraceOptions
    TraceLog::kInternalEchoToConsole = 1 << 3;

[nednguyen, 2014/07/30 16:51:13]

Done.

+const TraceLog::InternalTraceOptions TraceLog::kInternalEchoToConsole = 1
+                                                                          << 3;

[dsinclair, 2014/07/30 14:25:25]

Systrace?

[nednguyen, 2014/07/30 16:51:13]

No systrace as above.

+
 }  // namespace debug
 }  // namespace base