chromium/src
0
Fork 0
Code Activity

[PWA] IsInstallState migration part 2b

Browse Source 
Remove last IsInstallState call from the tests in the code
base.

(Discovered this one remaining instance with codesearch
today).

Bug: 381832688
Change-Id: I177e2d64c7028b2254bd0f14202ab1073c10db71
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6090193
Auto-Submit: Finnur Thorarinsson <finnur@chromium.org>
Commit-Queue: Finnur Thorarinsson <finnur@chromium.org>
Reviewed-by: Dibyajyoti Pal <dibyapal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1395841}
This commit is contained in:
Finnur Thorarinsson
2024-12-13 02:20:07 -08:00
committed by Chromium LUCI CQ
parent 6c1aefb96f
commit a0e7b80189
1 changed files with 4 additions and 8 deletions

12
chrome/browser/web_applications/commands/install_from_sync_command_browsertest.cc

@ -63,14 +63,10 @@ IN_PROC_BROWSER_TEST_F(InstallFromSyncCommandTest, SimpleInstall) {
loop.Quit();
})));
loop.Run();
EXPECT_TRUE(provider->registrar_unsafe().IsInstallState(
id, {proto::InstallState::SUGGESTED_FROM_ANOTHER_DEVICE,
proto::InstallState::INSTALLED_WITHOUT_OS_INTEGRATION,
proto::InstallState::INSTALLED_WITH_OS_INTEGRATION}));
EXPECT_EQ(AreAppsLocallyInstalledBySync(),
provider->registrar_unsafe().IsInstallState(
id, {proto::INSTALLED_WITHOUT_OS_INTEGRATION,
proto::INSTALLED_WITH_OS_INTEGRATION}));
EXPECT_EQ(provider->registrar_unsafe().GetInstallState(id),
AreAppsLocallyInstalledBySync()
? proto::INSTALLED_WITH_OS_INTEGRATION
: proto::SUGGESTED_FROM_ANOTHER_DEVICE);
SkColor icon_color =
IconManagerReadAppIconPixel(provider->icon_manager(), id, 96);