Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Unified Diff: chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc

Issue 2601323002: arc: Handle default app not availble case. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
diff --git a/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc b/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
index 4fd6dc885acc1d85d518d4928550dd1e006cd9d8..3ce5ca99a766b6cc356db386dbddc82513351cf6 100644
--- a/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
+++ b/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
@@ -227,13 +227,13 @@ class ArcAppLauncherBrowserTest : public ExtensionBrowserTest {
app_host()->OnPackageRemoved(package_name);
}
- void SendInstallationStarted() {
- app_host()->OnInstallationStarted();
+ void SendInstallationStarted(const std::string& package_name) {
+ app_host()->OnInstallationStarted(package_name);
base::RunLoop().RunUntilIdle();
}
- void SendInstallationFinished() {
- app_host()->OnInstallationFinished();
+ void SendInstallationFinished(const std::string& package_name, bool success) {
+ app_host()->OnInstallationFinished(package_name, success);
base::RunLoop().RunUntilIdle();
}
@@ -417,11 +417,11 @@ IN_PROC_BROWSER_TEST_F(ArcAppLauncherBrowserTest, AppListShown) {
EXPECT_FALSE(app_list_service->IsAppListVisible());
- SendInstallationStarted();
- SendInstallationStarted();
+ SendInstallationStarted(kTestAppPackage);
+ SendInstallationStarted(kTestAppPackage2);
// New package is available. Show app list.
- SendInstallationFinished();
+ SendInstallationFinished(kTestAppPackage, true);
InstallTestApps(kTestAppPackage, false);
SendPackageAdded(kTestAppPackage, true);
EXPECT_TRUE(app_list_service->IsAppListVisible());
@@ -435,14 +435,14 @@ IN_PROC_BROWSER_TEST_F(ArcAppLauncherBrowserTest, AppListShown) {
// Install next package from batch. Next new package is available.
// Don't show app list.
- SendInstallationFinished();
+ SendInstallationFinished(kTestAppPackage2, true);
InstallTestApps(kTestAppPackage2, false);
SendPackageAdded(kTestAppPackage2, true);
EXPECT_FALSE(app_list_service->IsAppListVisible());
// Run next installation batch. App list should be shown again.
- SendInstallationStarted();
- SendInstallationFinished();
+ SendInstallationStarted(kTestAppPackage3);
+ SendInstallationFinished(kTestAppPackage3, true);
InstallTestApps(kTestAppPackage3, false);
SendPackageAdded(kTestAppPackage3, true);
EXPECT_TRUE(app_list_service->IsAppListVisible());

Powered by Google App Engine
This is Rietveld 408576698