Check for CloseHandle failures even when not debugging

base/win/pe_image_test.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 <windows.h>
 
 #include <cfgmgr32.h>
 #include <shellapi.h>
 
 extern "C" {
 
 __declspec(dllexport) void ExportFunc1() {
   // Call into user32.dll.
   HWND dummy = GetDesktopWindow();
   SetWindowTextA(dummy, "dummy");
 }
 
 __declspec(dllexport) void ExportFunc2() {
   // Call into cfgmgr32.dll.
   CM_MapCrToWin32Err(CR_SUCCESS, ERROR_SUCCESS);
 
   // Call into shell32.dll.
   SHFILEOPSTRUCT file_operation = {0};
   SHFileOperation(&file_operation);
 
   // Call into kernel32.dll.
   HANDLE h = CreateEvent(NULL, FALSE, FALSE, NULL);
-  CloseHandle(h);
+  CHECK(::CloseHandle(h));

[rvargas (doing something else), 2015/09/16 02:09:12]

This is never executed, so I would not change this. (and besides, this would add
a dependency on base).

[brucedawson, 2015/09/16 18:29:00]

Ignore this change. I'm reverting it. The CHECK doesn't compile and I don't
think the code is run.

[brucedawson, 2015/09/16 18:29:00]

Done.

 }
 
 }  // extern "C"