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

Unified Diff: sandbox/win/src/policy_target_test.cc

Issue 2130753002: Made setting lowbox token a warning. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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/src/policy_target_test.cc
diff --git a/sandbox/win/src/policy_target_test.cc b/sandbox/win/src/policy_target_test.cc
index 71054abd3d5ab50f0e7f75930251c7b67ccecd67..7e3076b09509d8e9733738b7fef76f8c5d64f50c 100644
--- a/sandbox/win/src/policy_target_test.cc
+++ b/sandbox/win/src/policy_target_test.cc
@@ -238,6 +238,7 @@ TEST(PolicyTargetTest, DesktopPolicy) {
// Launch the app.
ResultCode result = SBOX_ALL_OK;
+ ResultCode warning_result = SBOX_ALL_OK;
base::win::ScopedProcessInformation target;
TargetPolicy* policy = broker->CreatePolicy();
@@ -245,7 +246,7 @@ TEST(PolicyTargetTest, DesktopPolicy) {
policy->SetTokenLevel(USER_INTERACTIVE, USER_LOCKDOWN);
PROCESS_INFORMATION temp_process_info = {};
result = broker->SpawnTarget(prog_name, arguments.c_str(), policy,
- &temp_process_info);
+ &warning_result, &temp_process_info);
base::string16 desktop_name = policy->GetAlternateDesktop();
policy->Release();
@@ -302,6 +303,7 @@ TEST(PolicyTargetTest, WinstaPolicy) {
// Launch the app.
ResultCode result = SBOX_ALL_OK;
+ ResultCode warning_result = SBOX_ALL_OK;
base::win::ScopedProcessInformation target;
TargetPolicy* policy = broker->CreatePolicy();
@@ -309,7 +311,7 @@ TEST(PolicyTargetTest, WinstaPolicy) {
policy->SetTokenLevel(USER_INTERACTIVE, USER_LOCKDOWN);
PROCESS_INFORMATION temp_process_info = {};
result = broker->SpawnTarget(prog_name, arguments.c_str(), policy,
- &temp_process_info);
+ &warning_result, &temp_process_info);
base::string16 desktop_name = policy->GetAlternateDesktop();
policy->Release();
@@ -382,12 +384,13 @@ TEST(PolicyTargetTest, ShareHandleTest) {
// Launch the app.
ResultCode result = SBOX_ALL_OK;
+ ResultCode warning_result = SBOX_ALL_OK;
base::win::ScopedProcessInformation target;
policy->SetTokenLevel(USER_INTERACTIVE, USER_LOCKDOWN);
PROCESS_INFORMATION temp_process_info = {};
result = broker->SpawnTarget(prog_name, arguments.c_str(), policy,
- &temp_process_info);
+ &warning_result, &temp_process_info);
policy->Release();
EXPECT_EQ(SBOX_ALL_OK, result);

Powered by Google App Engine
This is Rietveld 408576698