Revert of Enable handle verifier for tests and add some tests.

base/debug/close_handle_hook_win.cc

 // Copyright 2015 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/debug/close_handle_hook_win.h"
 
 #include <Windows.h>
 #include <psapi.h>
 #include <stddef.h>
 
 #include <algorithm>
 #include <vector>
 
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/win/iat_patch_function.h"
 #include "base/win/pe_image.h"
 #include "base/win/scoped_handle.h"
-#include "base/win/windows_version.h"
 #include "build/build_config.h"
 
 namespace {
 
 typedef BOOL (WINAPI* CloseHandleType) (HANDLE handle);
 
 typedef BOOL (WINAPI* DuplicateHandleType)(HANDLE source_process,
                                            HANDLE source_handle,
                                            HANDLE target_process,
                                            HANDLE* target_handle,
@@ skipping 222 matching lines @@
   returned = std::min(kSize, returned);
 
   for (DWORD current = 0; current < returned; current++) {
     hooks->AddIATPatch(modules[current]);
   }
 }
 
 }  // namespace
 
 void InstallHandleHooks() {
-#if !defined(DISABLE_HANDLE_VERIFIER_HOOKS)
-#if defined(_DEBUG)
-  // Handle hooks cause shutdown asserts in Debug on Windows 7. crbug.com/571304
-  if (base::win::GetVersion() < base::win::VERSION_WIN8)
-    return;
-#endif
   HandleHooks* hooks = g_hooks.Pointer();
 
   // Performing EAT interception first is safer in the presence of other
   // threads attempting to call CloseHandle.
   hooks->AddEATPatch();
   PatchLoadedModules(hooks);
-#endif
 }
 
 void RemoveHandleHooks() {
   // We are partching all loaded modules without forcing them to stay in memory,
   // removing patches is not safe.
 }
 
 }  // namespace debug
 }  // namespace base