Have all trace points emit to ETW.

base/debug/trace_event.h

Index: base/debug/trace_event.h
diff --git a/base/debug/trace_event.h b/base/debug/trace_event.h
index 43eb09c4771318528290d38d1ab0b28565da0833..87a6ba57f190306715f7781531ce12700b6ad1c2 100644
--- a/base/debug/trace_event.h
+++ b/base/debug/trace_event.h
@@ -907,9 +907,10 @@ TRACE_EVENT_API_CLASS_EXPORT extern \
 // definitions.
 
 // Phase indicates the nature of an event entry. E.g. part of a begin/end pair.
-#define TRACE_EVENT_PHASE_BEGIN    ('B')
-#define TRACE_EVENT_PHASE_END      ('E')
-#define TRACE_EVENT_PHASE_INSTANT  ('i')
+#define TRACE_EVENT_PHASE_BEGIN       ('B')
+#define TRACE_EVENT_PHASE_END         ('E')
+#define TRACE_EVENT_PHASE_INSTANT     ('i')
+#define TRACE_EVENT_PHASE_INSTANT_OLD ('I')  // Used by third party libraries.

[dsinclair, 2013/09/09 19:57:46]

What's the benefit of listing this here? The 'I' is historic and should not be
used going forward. Consumers should always use the TRACE_EVENT_PHASE_INSTANT.

[fdoray, 2013/09/16 00:51:03]

I need the value to handle events from external libraries correctly. I moved the
definition in EtwEventTypeFromInternalEventType() (trace_event_win.cc).

On 2013/09/09 19:57:46, dsinclair wrote:

 #define TRACE_EVENT_PHASE_ASYNC_BEGIN ('S')
 #define TRACE_EVENT_PHASE_ASYNC_STEP  ('T')
 #define TRACE_EVENT_PHASE_ASYNC_END   ('F')