Commit 21e29482 authored by Giovanni Ortuño Urquidi's avatar Giovanni Ortuño Urquidi Committed by Chromium LUCI CQ

desktop-pwas: Remove WebAppUrlLoader from PendingAppManager unit tests.

WebAppURLLoader is now an implementation detail of PendingAppInstallTask.
PendingAppManager unit test shouldn't need to set a fake WebAppUrlLoader
to control the timing of the test install tasks.

This CL removes WebAppUrlLoader from PendingAppManager unit tests in
favor of a "Task Manager" test class that tests can use to control the
timing of test tasks.

Bug: 1158722
Change-Id: If1d36c4e2ddd400f144b80df456a3ab7f4d58738
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2596488
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Reviewed-by: default avatarJiewei Qian  <qjw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#839705}
parent ee0cc24d
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