Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(690)

Unified Diff: sandbox/win/tests/integration_tests/hooking_win_proc.cc

Issue 2944493002: [Windows Sandbox Tests] Process Mitigations. (Closed)
Patch Set: Code review fixes, part 1. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sandbox/win/tests/integration_tests/hooking_win_proc.cc
diff --git a/sandbox/win/tests/integration_tests/hooking_win_proc.cc b/sandbox/win/tests/integration_tests/hooking_win_proc.cc
index 3c6770abff969a12287b180b25f13d504c673a50..e9d2d01f9be33c10c0738a04f558066382214ea1 100644
--- a/sandbox/win/tests/integration_tests/hooking_win_proc.cc
+++ b/sandbox/win/tests/integration_tests/hooking_win_proc.cc
@@ -2,10 +2,17 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "integration_tests_common.h"
-
#include <windows.h>
+#include "hooking_win_proc.h"
+
+namespace {
+
+constexpr wchar_t g_winproc_class_name[] = L"myWindowClass";
Will Harris 2017/06/28 12:27:55 declare these closest to use, so that would be ins
penny 2017/06/28 22:33:33 Done.
+constexpr wchar_t g_winproc_window_name[] = L"ChromeMitigationTests";
+
+} // namespace
+
LRESULT CALLBACK WndProc(HWND window,
UINT message,
WPARAM w_param,
@@ -34,7 +41,8 @@ int WINAPI WinMain(HINSTANCE instance,
LPSTR cmd_line,
int cmd_show) {
// The parent process should have set up this named event already.
- HANDLE event = ::OpenEventW(EVENT_MODIFY_STATE, FALSE, g_winproc_event);
+ HANDLE event = ::OpenEventW(EVENT_MODIFY_STATE, FALSE,
+ hooking_win_proc::g_winproc_event);
if (event == NULL || event == INVALID_HANDLE_VALUE)
return 1;

Powered by Google App Engine
This is Rietveld 408576698