Commit c8c85892 authored by Leo Zhang's avatar Leo Zhang Committed by Commit Bot

Correct the sandbox mapping for IME service.

When WithSandboxType is deprecated, a wrong Mojo interface was set for
ImeService launching, which invalidates the "ime" sandbox. And
ImeService's sandbox is set to "utility" by default always.


BUG=b:161958233
TEST="--service-sandbox-type=ime" was set for the ime process.

Change-Id: I14d0d4141fe8fe684935153ca05dee6cdd86fe26
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320448Reviewed-by: default avatarAlex Gough <ajgo@chromium.org>
Commit-Queue: Leo Zhang <googleo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#792227}
parent dde1224a
......@@ -13,18 +13,18 @@
// require a non-utility sandbox can be added here. See
// ServiceProcessHost::Launch() for how these templates are consumed.
// chromeos::ime::mojom::InputEngineManager
// chromeos::ime::mojom::ImeService
namespace chromeos {
namespace ime {
namespace mojom {
class InputEngineManager;
class ImeService;
} // namespace mojom
} // namespace ime
} // namespace chromeos
template <>
inline sandbox::policy::SandboxType
content::GetServiceSandboxType<chromeos::ime::mojom::InputEngineManager>() {
content::GetServiceSandboxType<chromeos::ime::mojom::ImeService>() {
if (chromeos::features::IsImeSandboxEnabled())
return sandbox::policy::SandboxType::kIme;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment