Chromium Code Reviews| Index: sandbox/win/tests/common/controller.cc |
| diff --git a/sandbox/win/tests/common/controller.cc b/sandbox/win/tests/common/controller.cc |
| index 6e45e0ef6c5ac78a41797133f5a58580b5fbb03c..0d2973f045afeb464995700be674849963f37fbd 100644 |
| --- a/sandbox/win/tests/common/controller.cc |
| +++ b/sandbox/win/tests/common/controller.cc |
| @@ -136,15 +136,12 @@ void TestRunner::Init(JobLevel job_level, |
| } |
| TargetPolicy* TestRunner::GetPolicy() { |
| - return policy_; |
| + return policy_.get(); |
| } |
| TestRunner::~TestRunner() { |
| if (target_process_.IsValid() && kill_on_destruction_) |
| ::TerminateProcess(target_process_.Get(), 0); |
| - |
| - if (policy_) |
| - policy_->Release(); |
| } |
| bool TestRunner::AddRule(TargetPolicy::SubSystem subsystem, |
| @@ -242,10 +239,8 @@ int TestRunner::InternalRunTest(const wchar_t* command) { |
| result = broker_->SpawnTarget(prog_name, arguments.c_str(), policy_, |
| &warning_result, &last_error, &target); |
| } |
| - if (release_policy_in_run_) { |
| - policy_->Release(); |
| + if (release_policy_in_run_) |
|
Will Harris
2017/01/24 19:47:32
note: new change
|
| policy_ = nullptr; |
| - } |
| if (SBOX_ALL_OK != result) |
| return SBOX_TEST_FAILED_TO_RUN_TEST; |