Commit 28e28546 authored by Kenichi Ishibashi's avatar Kenichi Ishibashi Committed by Commit Bot

S13nServiceWorker: Pass SharedWorkerServiceImplTest

When S13nServiceWorker is enabled these tests requre an instance
of URLLoaderFactory. Add a mock URLLoaderFactory and set them in
tests. This makes tests pass.

Bug: 860361
Change-Id: I2cfa371c1c1076d895398686b3df36340664e30b
Reviewed-on: https://chromium-review.googlesource.com/1152337Reviewed-by: default avatarKinuko Yasuda <kinuko@chromium.org>
Reviewed-by: default avatarMatt Falkenhagen <falken@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579243}
parent e5c79f50
...@@ -57,6 +57,7 @@ MockRenderProcessHost::MockRenderProcessHost(BrowserContext* browser_context) ...@@ -57,6 +57,7 @@ MockRenderProcessHost::MockRenderProcessHost(BrowserContext* browser_context)
child_identity_(mojom::kRendererServiceName, child_identity_(mojom::kRendererServiceName,
BrowserContext::GetServiceUserIdFor(browser_context), BrowserContext::GetServiceUserIdFor(browser_context),
base::StringPrintf("%d", id_)), base::StringPrintf("%d", id_)),
url_loader_factory_(nullptr),
weak_ptr_factory_(this) { weak_ptr_factory_(this) {
// Child process security operations can't be unit tested unless we add // Child process security operations can't be unit tested unless we add
// ourselves as an existing child process. // ourselves as an existing child process.
......
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