Add test for multithreaded ActiveVerifier behavior.

base/win/scoped_handle_test_dll.h

Index: base/win/scoped_handle_test_dll.h
diff --git a/base/win/scoped_handle_test_dll.h b/base/win/scoped_handle_test_dll.h
new file mode 100644
index 0000000000000000000000000000000000000000..f6126dd9a5d26e5e6bd40e3b2b01c510ca81bed1
--- /dev/null
+++ b/base/win/scoped_handle_test_dll.h
@@ -0,0 +1,26 @@
+// Copyright (c) 2016 The Chromium Authors. All rights reserved.

[scottmg, 2016/03/11 19:02:18]

No (c).

https://www.chromium.org/developers/coding-style#TOC-File-headers

[Will Harris, 2016/03/11 20:35:32]

bah! I chose the wrong file to copy and paste. And I felt so proud of myself
that I (for once) remembered to change the date to 2016.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef BASE_WIN_SCOPED_HANDLE_TEST_DLL_H_
+#define BASE_WIN_SCOPED_HANDLE_TEST_DLL_H_
+
+namespace base {
+namespace win {
+namespace testing {
+
+enum class Result {

[scottmg, 2016/03/11 19:02:18]

base::win::testing::Result seems a bit too generic of a name.

[Will Harris, 2016/03/11 20:35:32]

base::win::testing::ScopedHandleTestResult okay? I can tell you don't like my
enum. maybe once the test is fully passing I can change it to a bool?

+  SUCCESS = 0,
+  ERROR_COULD_NOT_CREATE_EVENT,

[scottmg, 2016/03/11 19:02:18]

You also don't seem to be using the great expanse of possible return values in
the test-checking code, so maybe just return a bool instead.

[Will Harris, 2016/03/11 20:35:32]

it's nice to know which error happened when it happens, but sure for testing I
suppose it doens't matter.

+  ERROR_COULD_NOT_GET_HANDLE_VERIFIER_MODULE,
+  ERROR_COULD_NOT_GET_MY_MODULE,
+  ERROR_WRONG_MODULE
+};
+
+extern "C" Result __declspec(dllexport) RunTest();
+
+}  // namespace testing
+}  // namespace win
+}  // namespace base
+
+#endif  // BASE_WIN_SCOPED_HANDLE_TEST_DLL_H_