Fixing VerifyLoadedModules not freeing the loaded dll.

GetModuleHandleEx was incrementing the module ref count so
the library was not freed at the end of the test. Another
unrelated unit test was expecting the dll to not be loaded.

Using GetModuleHandle now to avoid the ref count increment.

BUG=401854
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290025

[pmonette_google.com, 2014-08-14 21:32:30]

PTAL

[robertshield, 2014-08-15 01:16:34]

LGTM with two changes:

1) Please reference the bug for the rest of the verification work in the CL
description. 
2) Below:

https://codereview.chromium.org/471223002/diff/1/chrome/browser/safe_browsing...
File chrome/browser/safe_browsing/environment_data_collection_win_unittest.cc
(right):

https://codereview.chromium.org/471223002/diff/1/chrome/browser/safe_browsing...
chrome/browser/safe_browsing/environment_data_collection_win_unittest.cc:183:
HMODULE module_handle = GetModuleHandle(safe_browsing::kTestDllNames[0]);
Nice find! Instead of this, please use GetModuleHandleEx with
GET_MODULE_HANDLE_EX_FLAG_UNCHANGED_REFCOUNT as the first param to make it
explicit that we do not desire to change the refcount.

[pmonette_google.com, 2014-08-15 18:28:25]

The CQ bit was checked by pmonette@google.com

[commit-bot: I haz the power, 2014-08-15 18:32:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/pmonette@google.com/471223002/40001

[commit-bot: I haz the power, 2014-08-15 21:58:30]

Committed patchset #3 (40001) as 290025