Have all trace points emit to ETW.

base/debug/trace_event_win.cc

Index: base/debug/trace_event_win.cc
diff --git a/base/debug/trace_event_win.cc b/base/debug/trace_event_win.cc
index d5a21f49268f3b386a94e6d4d06d1fb57db1faeb..5caca343389516be44867e1f94878c8363a8188e 100644
--- a/base/debug/trace_event_win.cc
+++ b/base/debug/trace_event_win.cc
@@ -43,7 +43,7 @@ bool TraceEventETWProvider::StartTracing() {
 void TraceEventETWProvider::TraceEvent(const char* name,
                                        size_t name_len,
                                        char type,
-                                       const void* id,
+                                       unsigned long long id,
                                        const char* extra,
                                        size_t extra_len) {
   // Make sure we don't touch NULL.
@@ -97,7 +97,7 @@ void TraceEventETWProvider::TraceEvent(const char* name,
 void TraceEventETWProvider::Trace(const char* name,
                                   size_t name_len,
                                   char type,
-                                  const void* id,
+                                  unsigned long long id,

[chrisha, 2013/09/04 13:32:09]

Why are we changing the type of this?

[fdoray, 2013/09/05 20:10:31]

I wanted to avoid an extra type conversion: |id| comes from
TraceLog::AddTraceEventWithThreadIdAndTimestamp, which uses an "unsigned long
long". |id| can also come from the TRACE_EVENT_*_ETW macros (via
base::debug::TraceLog::AddTraceEventEtw), but I understand from crbug.com/81584
that these macros will disappear.

Sawbuck had trouble displaying |extra| when I used an "unsigned long long" so I
will keep a "const void*" for now.

On 2013/09/04 13:32:09, chrisha wrote:

                                   const char* extra,
                                   size_t extra_len) {
   TraceEventETWProvider* provider = TraceEventETWProvider::GetInstance();