Chromium Code Reviews| Index: sandbox/win/src/process_mitigations_test.cc |
| diff --git a/sandbox/win/src/process_mitigations_test.cc b/sandbox/win/src/process_mitigations_test.cc |
| index 4d2e9c6e9562a6abdd3488afaa2a4636cc893fac..4b3d3956973d08b9888f5a030f77aa49ec5610f8 100644 |
| --- a/sandbox/win/src/process_mitigations_test.cc |
| +++ b/sandbox/win/src/process_mitigations_test.cc |
| @@ -165,9 +165,9 @@ SBOX_TESTS_COMMAND int CheckDep(int argc, wchar_t **argv) { |
| return SBOX_TEST_THIRD_ERROR; |
| } |
| - static const int MEM_EXECUTE_OPTION_ENABLE = 1; |
| + // static const int MEM_EXECUTE_OPTION_ENABLE = 1; |
|
Nico
2015/07/02 17:34:26
?
Matt Giuca
2015/07/03 03:46:29
Well I thought that since this seems to represent
Nico
2015/07/03 04:11:15
As-is, it looks like code that was left in acciden
Matt Giuca
2015/07/03 04:33:45
jschuh: Can I get your quick opinion on this. You
|
| static const int MEM_EXECUTE_OPTION_DISABLE = 2; |
| - static const int MEM_EXECUTE_OPTION_ATL7_THUNK_EMULATION = 4; |
| + // static const int MEM_EXECUTE_OPTION_ATL7_THUNK_EMULATION = 4; |
|
Nico
2015/07/02 17:34:26
?
jschuh
2015/07/06 22:16:18
Yeah, looks like dead constants. lgtm on removing
Matt Giuca
2015/07/07 04:52:17
Well they were dead the moment they were added, so
|
| static const int MEM_EXECUTE_OPTION_PERMANENT = 8; |
| dep_flags &= 0xff; |