| Index: chrome/browser/chromeos/app_mode/arc/arc_kiosk_app_manager_browsertest.cc
|
| diff --git a/chrome/browser/chromeos/app_mode/arc/arc_kiosk_app_manager_browsertest.cc b/chrome/browser/chromeos/app_mode/arc/arc_kiosk_app_manager_browsertest.cc
|
| index f334af77a0601fb79ae7536528f8e10a695a98e3..a35c94b3ca035f376d4de43672d977afbd25b4cc 100644
|
| --- a/chrome/browser/chromeos/app_mode/arc/arc_kiosk_app_manager_browsertest.cc
|
| +++ b/chrome/browser/chromeos/app_mode/arc/arc_kiosk_app_manager_browsertest.cc
|
| @@ -158,12 +158,13 @@ IN_PROC_BROWSER_TEST_F(ArcKioskAppManagerTest, Basic) {
|
| waiter.Wait();
|
| EXPECT_TRUE(waiter.was_notified());
|
|
|
| - ArcKioskAppManager::ArcKioskApps apps = manager()->GetAllApps();
|
| + ArcKioskAppManager::Apps apps;
|
| + manager()->GetAllApps(&apps);
|
| ASSERT_EQ(2u, apps.size());
|
| - ASSERT_EQ(app1, apps[0].app_info());
|
| - ASSERT_EQ(app2, apps[1].app_info());
|
| - ASSERT_EQ(app1.package_name(), apps[0].name());
|
| - ASSERT_EQ(app2.display_name(), apps[1].name());
|
| + ASSERT_EQ(app1.package_name(), apps[0]->app_id());
|
| + ASSERT_EQ(app2.package_name(), apps[1]->app_id());
|
| + ASSERT_EQ(app1.package_name(), apps[0]->name());
|
| + ASSERT_EQ(app2.display_name(), apps[1]->name());
|
| EXPECT_FALSE(manager()->GetAutoLaunchAccountId().is_valid());
|
| }
|
|
|
| @@ -177,15 +178,16 @@ IN_PROC_BROWSER_TEST_F(ArcKioskAppManagerTest, Basic) {
|
| EXPECT_TRUE(waiter.was_notified());
|
|
|
| EXPECT_TRUE(manager()->GetAutoLaunchAccountId().is_valid());
|
| - ASSERT_EQ(2u, manager()->GetAllApps().size());
|
|
|
| - ArcKioskAppManager::ArcKioskApps apps = manager()->GetAllApps();
|
| - ASSERT_EQ(app1, apps[0].app_info());
|
| - ASSERT_EQ(app2, apps[1].app_info());
|
| - ASSERT_EQ(app1.package_name(), apps[0].name());
|
| - ASSERT_EQ(app2.display_name(), apps[1].name());
|
| + ArcKioskAppManager::Apps apps;
|
| + manager()->GetAllApps(&apps);
|
| + ASSERT_EQ(2u, apps.size());
|
| + ASSERT_EQ(app1.package_name(), apps[0]->app_id());
|
| + ASSERT_EQ(app2.package_name(), apps[1]->app_id());
|
| + ASSERT_EQ(app1.package_name(), apps[0]->name());
|
| + ASSERT_EQ(app2.display_name(), apps[1]->name());
|
| EXPECT_TRUE(manager()->GetAutoLaunchAccountId().is_valid());
|
| - ASSERT_EQ(apps[1].account_id(), manager()->GetAutoLaunchAccountId());
|
| + ASSERT_EQ(apps[1]->account_id(), manager()->GetAutoLaunchAccountId());
|
| }
|
|
|
| // Create a new list of apps, where there is no app2 (is auto launch now),
|
| @@ -199,12 +201,13 @@ IN_PROC_BROWSER_TEST_F(ArcKioskAppManagerTest, Basic) {
|
| waiter.Wait();
|
| EXPECT_TRUE(waiter.was_notified());
|
|
|
| - ArcKioskAppManager::ArcKioskApps apps = manager()->GetAllApps();
|
| + ArcKioskAppManager::Apps apps;
|
| + manager()->GetAllApps(&apps);
|
| ASSERT_EQ(2u, apps.size());
|
| - ASSERT_EQ(app1, apps[0].app_info());
|
| - ASSERT_EQ(app3, apps[1].app_info());
|
| - ASSERT_EQ(app1.package_name(), apps[0].name());
|
| - ASSERT_EQ(app3.package_name(), apps[1].name());
|
| + ASSERT_EQ(app1.package_name(), apps[0]->app_id());
|
| + ASSERT_EQ(app3.package_name(), apps[1]->app_id());
|
| + ASSERT_EQ(app1.package_name(), apps[0]->name());
|
| + ASSERT_EQ(app3.package_name(), apps[1]->name());
|
| // Auto launch app must be reset.
|
| EXPECT_FALSE(manager()->GetAutoLaunchAccountId().is_valid());
|
| }
|
| @@ -217,7 +220,9 @@ IN_PROC_BROWSER_TEST_F(ArcKioskAppManagerTest, Basic) {
|
| waiter.Wait();
|
| EXPECT_TRUE(waiter.was_notified());
|
|
|
| - ASSERT_EQ(0u, manager()->GetAllApps().size());
|
| + ArcKioskAppManager::Apps apps;
|
| + manager()->GetAllApps(&apps);
|
| + ASSERT_EQ(0u, apps.size());
|
| EXPECT_FALSE(manager()->GetAutoLaunchAccountId().is_valid());
|
| }
|
| }
|
|
|