Enable handle verifier for tests and add some tests.

base/test/test_suite.cc

 // 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.
 
 #include "base/test/test_suite.h"
 
 #include "base/at_exit.h"
 #include "base/base_paths.h"
 #include "base/base_switches.h"
 #include "base/bind.h"
@@ skipping 22 matching lines @@
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/multiprocess_func_list.h"
 
 #if defined(OS_MACOSX)
 #include "base/mac/scoped_nsautorelease_pool.h"
 #if defined(OS_IOS)
 #include "base/test/test_listener_ios.h"
 #endif  // OS_IOS
 #endif  // OS_MACOSX
 
-#if !defined(OS_WIN)
+#if defined(OS_WIN)
+#include "base/debug/close_handle_hook_win.h"
+#include "base/win/windows_version.h"
+#else
 #include "base/i18n/rtl.h"
 #if !defined(OS_IOS)
 #include "base/strings/string_util.h"
 #include "third_party/icu/source/common/unicode/uloc.h"
 #endif
 #endif
 
 #if defined(OS_ANDROID)
 #include "base/test/test_support_android.h"
 #endif
@@ skipping 256 matching lines @@
   settings.log_file = log_filename.value().c_str();
   settings.delete_old = logging::DELETE_OLD_LOG_FILE;
   logging::InitLogging(settings);
   // We want process and thread IDs because we may have multiple processes.
   // Note: temporarily enabled timestamps in an effort to catch bug 6361.
   logging::SetLogItems(true, true, true, true);
 #endif  // else defined(OS_ANDROID)
 
   CHECK(debug::EnableInProcessStackDumping());
 #if defined(OS_WIN)
+#if defined(_DEBUG)

[scottmg, 2016/02/01 16:45:31]

It's sort of weird that sometimes it's disabled here, and sometimes in the body
of the function.

[Will Harris, 2016/02/01 19:16:12]

Done.

Another option is expose something like base::debug::AreHandleHooksInPlace() but
since this is only going to be used in very few places I'm not sure it's worth
it.

+  // Handle hooks cause shutdown asserts in Debug on Windows 7. crbug.com/571304
+  if (base::win::GetVersion() >= base::win::VERSION_WIN8)
+    base::debug::InstallHandleHooks();
+#else
+  base::debug::InstallHandleHooks();
+#endif
+
   RouteStdioToConsole(true);
   // Make sure we run with high resolution timer to minimize differences
   // between production code and test code.
   Time::EnableHighResolutionTimer(true);
 #endif  // defined(OS_WIN)
 
   // In some cases, we do not want to see standard error dialogs.
   if (!debug::BeingDebugged() &&
       !CommandLine::ForCurrentProcess()->HasSwitch("show-error-dialogs")) {
     SuppressErrorDialogs();
@@ skipping 25 matching lines @@
 
   TestTimeouts::Initialize();
 
   trace_to_file_.BeginTracingFromCommandLineOptions();
 }
 
 void TestSuite::Shutdown() {
 }
 
 }  // namespace base