New test infrastructure for producing verbose logs in failing tests.

base/win/event_trace_provider.cc

Index: base/win/event_trace_provider.cc
diff --git a/base/win/event_trace_provider.cc b/base/win/event_trace_provider.cc
index 332fac32b50003e09008c59f2dd95ff223b7200f..8fcf67d3d701c1765f99aec66b028829181772d7 100644
--- a/base/win/event_trace_provider.cc
+++ b/base/win/event_trace_provider.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 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.
 //
@@ -84,12 +84,12 @@ ULONG EtwTraceProvider::Register() {
 }
 
 ULONG EtwTraceProvider::Unregister() {
+  // If a session is active, notify subclasses that it's going away.
+  if (session_handle_ != NULL)
+    DisableEvents();

[robertshield, 2012/03/09 16:58:04]

if session_handle_ is null, the level and flags no longer get reset, is this
intended?

[grt (UTC plus 2), 2012/03/09 17:55:25]

I don't see a way for session_handle_ to be NULL and the flags/level to not be
zero.  That aside, if session_handle_ is NULL, the level and flags are
meaningless.

+
   ULONG ret = ::UnregisterTraceGuids(registration_handle_);
 
-  // Make sure we don't log anything from here on.
-  enable_level_ = 0;
-  enable_flags_ = 0;
-  session_handle_ = NULL;
   registration_handle_ = NULL;
 
   return ret;