GitLab

dwpa: Modified PendingAppInstallTask to use a constructor provided InstallManager to support it as a test fixture.

Previously PendingAppInstallTask::ContinueWebAppInstall fetched the InstallManager via the WebAppProvider via the profile. This change passes the InstallManager via the constructor and avoids this lookup.

auto* provider = WebAppProviderBase::GetProviderBase(profile_);
provider->install_manager().InstallWebAppWithParams(

There will be a follow up CL where the system_web_app_manager_unittest.cc tests are refactored. This change to the pending_app_install_task is needed for the refactor.

Bug: 1082880
Change-Id: I038f0005325c77742d15ef3d5e1673903fdf7490
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2229013Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Commit-Queue: Lorne Mitchell <lomitch@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#774928}