Commit c96752a2 authored by proberge's avatar proberge Committed by Commit Bot

Shortcut parser test: use LoggedParserSandboxSetupHooks subclass

I didn't catch it while reviewing
https://chromium-review.googlesource.com/c/chromium/src/+/1948235/3/chrome/chrome_cleaner/parsers/shortcut_parser/broker/sandboxed_shortcut_parser_unittest.cc,
but the new LoggedParserSandboxSetupHooks class isn't used anywhere.

Change-Id: Iea07c34c57d13610f3e40df3697675da6c42733d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1956676
Commit-Queue: proberge <proberge@chromium.org>
Auto-Submit: proberge <proberge@chromium.org>
Reviewed-by: default avatarJoe Mason <joenotcharles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#723070}
parent 0b40c851
......@@ -64,9 +64,10 @@ class SandboxedShortcutParserTest : public base::MultiProcessTest {
ASSERT_TRUE(child_process_logger_.Initialize());
mojo_task_runner_ = MojoTaskRunner::Create();
ParserSandboxSetupHooks setup_hooks(
LoggedParserSandboxSetupHooks setup_hooks(
mojo_task_runner_.get(),
base::BindOnce([] { FAIL() << "Parser sandbox connection error"; }));
base::BindOnce([] { FAIL() << "Parser sandbox connection error"; }),
&child_process_logger_);
ResultCode result_code =
StartSandboxTarget(MakeCmdLine("SandboxedShortcutParserTargetMain"),
......
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