summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVitaly Buka <vitalybuka@chromium.org>2016-01-27 12:26:53 -0800
committerVitaly Buka <vitalybuka@chromium.org>2016-01-27 20:30:01 +0000
commit54362a9e01dc929f42b17a457deda8e1746b9cc7 (patch)
tree1c557036c119e60de0259727a4c0a7e5247f5853
parent8bccfe116d4530e944f03dae096e1e896594014c (diff)
downloadchromium_src-54362a9e01dc929f42b17a457deda8e1746b9cc7.zip
chromium_src-54362a9e01dc929f42b17a457deda8e1746b9cc7.tar.gz
chromium_src-54362a9e01dc929f42b17a457deda8e1746b9cc7.tar.bz2
Do not check result of SetAlternateDesktop
Fixed regression introduced by https://codereview.chromium.org/1415483008 Seems comment nearby was not noticed. BUG=581134 Review URL: https://codereview.chromium.org/1638773002 Cr-Commit-Position: refs/heads/master@{#371533} (cherry picked from commit 413e0b007b3f8d41d40915506e4e77a673d2e395) Review URL: https://codereview.chromium.org/1643713002 . Cr-Commit-Position: refs/branch-heads/2623@{#168} Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
-rw-r--r--chrome/service/service_utility_process_host.cc7
1 files changed, 4 insertions, 3 deletions
diff --git a/chrome/service/service_utility_process_host.cc b/chrome/service/service_utility_process_host.cc
index 46483ff..3ed91b4 100644
--- a/chrome/service/service_utility_process_host.cc
+++ b/chrome/service/service_utility_process_host.cc
@@ -67,9 +67,10 @@ class ServiceSandboxedProcessLauncherDelegate
ServiceSandboxedProcessLauncherDelegate() {}
bool PreSpawnTarget(sandbox::TargetPolicy* policy) override {
- // Service process may run as windows service and it fails to create a
- // window station.
- return policy->SetAlternateDesktop(false) == sandbox::SBOX_ALL_OK;
+ // Ignore result of SetAlternateDesktop. Service process may run as windows
+ // service and it fails to create a window station.
+ base::IgnoreResult(policy->SetAlternateDesktop(false));
+ return true;
}
private: