Chromium Code Reviews| Index: base/trace_event/trace_event.h |
| diff --git a/base/trace_event/trace_event.h b/base/trace_event/trace_event.h |
| index 7f73071de27b06c41c0083ce244e0798d81a2171..f996f9f4f54c572c697420db53ea321cc555bff5 100644 |
| --- a/base/trace_event/trace_event.h |
| +++ b/base/trace_event/trace_event.h |
| @@ -141,7 +141,7 @@ |
| // means, if the category for the event is disabled, the conversion will not |
| // happen. |
| // |
| -// class MyData : public base::debug::ConvertableToTraceFormat { |
| +// class MyData : public base::trace_event::ConvertableToTraceFormat { |
| // public: |
| // MyData() {} |
| // virtual void AppendAsTraceFormat(std::string* out) const override { |
| @@ -795,8 +795,8 @@ |
| #define INTERNAL_TRACE_EVENT_CATEGORY_GROUP_ENABLED_FOR_RECORDING_MODE() \ |
| UNLIKELY(*INTERNAL_TRACE_EVENT_UID(category_group_enabled) & \ |
| - (base::debug::TraceLog::ENABLED_FOR_RECORDING | \ |
| - base::debug::TraceLog::ENABLED_FOR_EVENT_CALLBACK)) |
| + (base::trace_event::TraceLog::ENABLED_FOR_RECORDING | \ |
| + base::trace_event::TraceLog::ENABLED_FOR_EVENT_CALLBACK)) |
| // Macro to efficiently determine if a given category group is enabled. |
| #define TRACE_EVENT_CATEGORY_GROUP_ENABLED(category_group, ret) \ |
| @@ -839,16 +839,16 @@ |
| // const unsigned char* |
| // TRACE_EVENT_API_GET_CATEGORY_GROUP_ENABLED(const char* category_group) |
| #define TRACE_EVENT_API_GET_CATEGORY_GROUP_ENABLED \ |
| - base::debug::TraceLog::GetCategoryGroupEnabled |
| + base::trace_event::TraceLog::GetCategoryGroupEnabled |
| // Get the number of times traces have been recorded. This is used to implement |
| // the TRACE_EVENT_IS_NEW_TRACE facility. |
| // unsigned int TRACE_EVENT_API_GET_NUM_TRACES_RECORDED() |
| #define TRACE_EVENT_API_GET_NUM_TRACES_RECORDED \ |
| - base::debug::TraceLog::GetInstance()->GetNumTracesRecorded |
| + base::trace_event::TraceLog::GetInstance()->GetNumTracesRecorded |
| // Add a trace event to the platform tracing system. |
| -// base::debug::TraceEventHandle TRACE_EVENT_API_ADD_TRACE_EVENT( |
| +// base::trace_event::TraceEventHandle TRACE_EVENT_API_ADD_TRACE_EVENT( |
| // char phase, |
| // const unsigned char* category_group_enabled, |
| // const char* name, |
| @@ -859,10 +859,11 @@ |
| // const unsigned long long* arg_values, |
| // unsigned char flags) |
| #define TRACE_EVENT_API_ADD_TRACE_EVENT \ |
| - base::debug::TraceLog::GetInstance()->AddTraceEvent |
| + base::trace_event::TraceLog::GetInstance()->AddTraceEvent |
| // Add a trace event to the platform tracing system. |
| -// base::debug::TraceEventHandle TRACE_EVENT_API_ADD_TRACE_EVENT_WITH_TIMESTAMP( |
| +// base::trace_event::TraceEventHandle |
| +// TRACE_EVENT_API_ADD_TRACE_EVENT_WITH_TIMESTAMP( |
| // char phase, |
| // const unsigned char* category_group_enabled, |
| // const char* name, |
| @@ -875,15 +876,16 @@ |
| // const unsigned long long* arg_values, |
| // unsigned char flags) |
| #define TRACE_EVENT_API_ADD_TRACE_EVENT_WITH_THREAD_ID_AND_TIMESTAMP \ |
| - base::debug::TraceLog::GetInstance()->AddTraceEventWithThreadIdAndTimestamp |
| + base::trace_event::TraceLog::GetInstance() \ |
| + ->AddTraceEventWithThreadIdAndTimestamp |
| // Set the duration field of a COMPLETE trace event. |
| // void TRACE_EVENT_API_UPDATE_TRACE_EVENT_DURATION( |
| // const unsigned char* category_group_enabled, |
| // const char* name, |
| -// base::debug::TraceEventHandle id) |
| +// base::trace_event::TraceEventHandle id) |
| #define TRACE_EVENT_API_UPDATE_TRACE_EVENT_DURATION \ |
| - base::debug::TraceLog::GetInstance()->UpdateTraceEventDuration |
| + base::trace_event::TraceLog::GetInstance()->UpdateTraceEventDuration |
| // Defines atomic operations used internally by the tracing system. |
| #define TRACE_EVENT_API_ATOMIC_WORD base::subtle::AtomicWord |
| @@ -956,11 +958,12 @@ TRACE_EVENT_API_CLASS_EXPORT extern \ |
| INTERNAL_TRACE_EVENT_GET_CATEGORY_INFO(category_group); \ |
| trace_event_internal::ScopedTracer INTERNAL_TRACE_EVENT_UID(tracer); \ |
| if (INTERNAL_TRACE_EVENT_CATEGORY_GROUP_ENABLED_FOR_RECORDING_MODE()) { \ |
| - base::debug::TraceEventHandle h = trace_event_internal::AddTraceEvent( \ |
| - TRACE_EVENT_PHASE_COMPLETE, \ |
| - INTERNAL_TRACE_EVENT_UID(category_group_enabled), \ |
| - name, trace_event_internal::kNoEventId, \ |
| - TRACE_EVENT_FLAG_NONE, ##__VA_ARGS__); \ |
| + base::trace_event::TraceEventHandle h = \ |
| + trace_event_internal::AddTraceEvent( \ |
| + TRACE_EVENT_PHASE_COMPLETE, \ |
| + INTERNAL_TRACE_EVENT_UID(category_group_enabled), name, \ |
| + trace_event_internal::kNoEventId, TRACE_EVENT_FLAG_NONE, \ |
| + ##__VA_ARGS__); \ |
| INTERNAL_TRACE_EVENT_UID(tracer).Initialize( \ |
| INTERNAL_TRACE_EVENT_UID(category_group_enabled), name, h); \ |
| } |
| @@ -1253,7 +1256,7 @@ static inline void SetTraceValue(const base::TimeTicks arg, |
| // pointers to the internal c_str and pass through to the tracing API, |
| // the arg_values must live throughout these procedures. |
| -static inline base::debug::TraceEventHandle |
| +static inline base::trace_event::TraceEventHandle |
| AddTraceEventWithThreadIdAndTimestamp( |
| char phase, |
| const unsigned char* category_group_enabled, |
| @@ -1263,7 +1266,8 @@ AddTraceEventWithThreadIdAndTimestamp( |
| const base::TimeTicks& timestamp, |
| unsigned char flags, |
| const char* arg1_name, |
| - const scoped_refptr<base::debug::ConvertableToTraceFormat>& arg1_val) { |
| + const scoped_refptr<base::trace_event::ConvertableToTraceFormat>& |
| + arg1_val) { |
| const int num_args = 1; |
| unsigned char arg_types[1] = { TRACE_VALUE_TYPE_CONVERTABLE }; |
| return TRACE_EVENT_API_ADD_TRACE_EVENT_WITH_THREAD_ID_AND_TIMESTAMP( |
| @@ -1272,7 +1276,7 @@ AddTraceEventWithThreadIdAndTimestamp( |
| } |
| template<class ARG1_TYPE> |
| -static inline base::debug::TraceEventHandle |
| +static inline base::trace_event::TraceEventHandle |
| AddTraceEventWithThreadIdAndTimestamp( |
| char phase, |
| const unsigned char* category_group_enabled, |
| @@ -1284,7 +1288,8 @@ AddTraceEventWithThreadIdAndTimestamp( |
| const char* arg1_name, |
| const ARG1_TYPE& arg1_val, |
| const char* arg2_name, |
| - const scoped_refptr<base::debug::ConvertableToTraceFormat>& arg2_val) { |
| + const scoped_refptr<base::trace_event::ConvertableToTraceFormat>& |
| + arg2_val) { |
| const int num_args = 2; |
| const char* arg_names[2] = { arg1_name, arg2_name }; |
| @@ -1293,7 +1298,8 @@ AddTraceEventWithThreadIdAndTimestamp( |
| SetTraceValue(arg1_val, &arg_types[0], &arg_values[0]); |
| arg_types[1] = TRACE_VALUE_TYPE_CONVERTABLE; |
| - scoped_refptr<base::debug::ConvertableToTraceFormat> convertable_values[2]; |
| + scoped_refptr<base::trace_event::ConvertableToTraceFormat> |
| + convertable_values[2]; |
| convertable_values[1] = arg2_val; |
| return TRACE_EVENT_API_ADD_TRACE_EVENT_WITH_THREAD_ID_AND_TIMESTAMP( |
| @@ -1302,7 +1308,7 @@ AddTraceEventWithThreadIdAndTimestamp( |
| } |
| template<class ARG2_TYPE> |
| -static inline base::debug::TraceEventHandle |
| +static inline base::trace_event::TraceEventHandle |
| AddTraceEventWithThreadIdAndTimestamp( |
| char phase, |
| const unsigned char* category_group_enabled, |
| @@ -1312,7 +1318,7 @@ AddTraceEventWithThreadIdAndTimestamp( |
| const base::TimeTicks& timestamp, |
| unsigned char flags, |
| const char* arg1_name, |
| - const scoped_refptr<base::debug::ConvertableToTraceFormat>& arg1_val, |
| + const scoped_refptr<base::trace_event::ConvertableToTraceFormat>& arg1_val, |
| const char* arg2_name, |
| const ARG2_TYPE& arg2_val) { |
| const int num_args = 2; |
| @@ -1324,7 +1330,8 @@ AddTraceEventWithThreadIdAndTimestamp( |
| arg_values[0] = 0; |
| SetTraceValue(arg2_val, &arg_types[1], &arg_values[1]); |
| - scoped_refptr<base::debug::ConvertableToTraceFormat> convertable_values[2]; |
| + scoped_refptr<base::trace_event::ConvertableToTraceFormat> |
| + convertable_values[2]; |
| convertable_values[0] = arg1_val; |
| return TRACE_EVENT_API_ADD_TRACE_EVENT_WITH_THREAD_ID_AND_TIMESTAMP( |
| @@ -1332,7 +1339,7 @@ AddTraceEventWithThreadIdAndTimestamp( |
| num_args, arg_names, arg_types, arg_values, convertable_values, flags); |
| } |
| -static inline base::debug::TraceEventHandle |
| +static inline base::trace_event::TraceEventHandle |
| AddTraceEventWithThreadIdAndTimestamp( |
| char phase, |
| const unsigned char* category_group_enabled, |
| @@ -1342,22 +1349,23 @@ AddTraceEventWithThreadIdAndTimestamp( |
| const base::TimeTicks& timestamp, |
| unsigned char flags, |
| const char* arg1_name, |
| - const scoped_refptr<base::debug::ConvertableToTraceFormat>& arg1_val, |
| + const scoped_refptr<base::trace_event::ConvertableToTraceFormat>& arg1_val, |
| const char* arg2_name, |
| - const scoped_refptr<base::debug::ConvertableToTraceFormat>& arg2_val) { |
| + const scoped_refptr<base::trace_event::ConvertableToTraceFormat>& |
| + arg2_val) { |
| const int num_args = 2; |
| const char* arg_names[2] = { arg1_name, arg2_name }; |
| unsigned char arg_types[2] = |
| { TRACE_VALUE_TYPE_CONVERTABLE, TRACE_VALUE_TYPE_CONVERTABLE }; |
| - scoped_refptr<base::debug::ConvertableToTraceFormat> convertable_values[2] = |
| - { arg1_val, arg2_val }; |
| + scoped_refptr<base::trace_event::ConvertableToTraceFormat> |
| + convertable_values[2] = {arg1_val, arg2_val}; |
| return TRACE_EVENT_API_ADD_TRACE_EVENT_WITH_THREAD_ID_AND_TIMESTAMP( |
| phase, category_group_enabled, name, id, thread_id, timestamp, |
| num_args, arg_names, arg_types, NULL, convertable_values, flags); |
| } |
| -static inline base::debug::TraceEventHandle |
| +static inline base::trace_event::TraceEventHandle |
| AddTraceEventWithThreadIdAndTimestamp( |
| char phase, |
| const unsigned char* category_group_enabled, |
| @@ -1371,7 +1379,7 @@ AddTraceEventWithThreadIdAndTimestamp( |
| kZeroNumArgs, NULL, NULL, NULL, NULL, flags); |
| } |
| -static inline base::debug::TraceEventHandle AddTraceEvent( |
| +static inline base::trace_event::TraceEventHandle AddTraceEvent( |
| char phase, |
| const unsigned char* category_group_enabled, |
| const char* name, |
| @@ -1384,7 +1392,7 @@ static inline base::debug::TraceEventHandle AddTraceEvent( |
| } |
| template<class ARG1_TYPE> |
| -static inline base::debug::TraceEventHandle |
| +static inline base::trace_event::TraceEventHandle |
| AddTraceEventWithThreadIdAndTimestamp( |
| char phase, |
| const unsigned char* category_group_enabled, |
| @@ -1405,7 +1413,7 @@ AddTraceEventWithThreadIdAndTimestamp( |
| } |
| template<class ARG1_TYPE> |
| -static inline base::debug::TraceEventHandle AddTraceEvent( |
| +static inline base::trace_event::TraceEventHandle AddTraceEvent( |
| char phase, |
| const unsigned char* category_group_enabled, |
| const char* name, |
| @@ -1421,7 +1429,7 @@ static inline base::debug::TraceEventHandle AddTraceEvent( |
| } |
| template<class ARG1_TYPE, class ARG2_TYPE> |
| -static inline base::debug::TraceEventHandle |
| +static inline base::trace_event::TraceEventHandle |
| AddTraceEventWithThreadIdAndTimestamp( |
| char phase, |
| const unsigned char* category_group_enabled, |
| @@ -1446,7 +1454,7 @@ AddTraceEventWithThreadIdAndTimestamp( |
| } |
| template<class ARG1_TYPE, class ARG2_TYPE> |
| -static inline base::debug::TraceEventHandle AddTraceEvent( |
| +static inline base::trace_event::TraceEventHandle AddTraceEvent( |
| char phase, |
| const unsigned char* category_group_enabled, |
| const char* name, |
| @@ -1478,7 +1486,7 @@ class TRACE_EVENT_API_CLASS_EXPORT ScopedTracer { |
| void Initialize(const unsigned char* category_group_enabled, |
| const char* name, |
| - base::debug::TraceEventHandle event_handle) { |
| + base::trace_event::TraceEventHandle event_handle) { |
| data_.category_group_enabled = category_group_enabled; |
| data_.name = name; |
| data_.event_handle = event_handle; |
| @@ -1494,7 +1502,7 @@ class TRACE_EVENT_API_CLASS_EXPORT ScopedTracer { |
| struct Data { |
| const unsigned char* category_group_enabled; |
| const char* name; |
| - base::debug::TraceEventHandle event_handle; |
| + base::trace_event::TraceEventHandle event_handle; |
| }; |
| Data* p_data_; |
| Data data_; |
| @@ -1509,13 +1517,13 @@ class TRACE_EVENT_API_CLASS_EXPORT ScopedTraceBinaryEfficient { |
| private: |
| const unsigned char* category_group_enabled_; |
| const char* name_; |
| - base::debug::TraceEventHandle event_handle_; |
| + base::trace_event::TraceEventHandle event_handle_; |
| }; |
| // This macro generates less code then TRACE_EVENT0 but is also |
| // slower to execute when tracing is off. It should generally only be |
| // used with code that is seldom executed or conditionally executed |
| -// when debugging. |
| +// when trace_eventging. |
| // For now the category_group must be "gpu". |
| #define TRACE_EVENT_BINARY_EFFICIENT0(category_group, name) \ |
| trace_event_internal::ScopedTraceBinaryEfficient \ |
| @@ -1555,7 +1563,7 @@ class TraceEventSamplingStateScope { |
| } // namespace trace_event_internal |
| namespace base { |
| -namespace debug { |
| +namespace trace_event { |
| template<typename IDType> class TraceScopedTrackableObject { |
| public: |
| @@ -1583,7 +1591,13 @@ template<typename IDType> class TraceScopedTrackableObject { |
| DISALLOW_COPY_AND_ASSIGN(TraceScopedTrackableObject); |
| }; |
| -} // namespace debug |
| -} // namespace base |
| +} // namespace trace_event |
| +} // namespace base |
| + |
| +namespace base { |
|
Primiano Tucci (use gerrit)
2015/01/27 14:34:10
Can you please add a TODO(ssid) which a refernce t
ssid
2015/01/27 15:56:45
Done.
|
| +namespace debug { |
| +using base::trace_event::TraceScopedTrackableObject; |
| +} // namespace debug |
| +} // namespace base |
| #endif /* BASE_TRACE_EVENT_TRACE_EVENT_H_ */ |