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

Unified Diff: chrome/browser/chromeos/login/kiosk_browsertest.cc

Issue 2576353002: Make AppWindowWaiter a common test support (Closed)
Patch Set: nits 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/chromeos/login/kiosk_browsertest.cc
diff --git a/chrome/browser/chromeos/login/kiosk_browsertest.cc b/chrome/browser/chromeos/login/kiosk_browsertest.cc
index b2767924b6981b6480c65595fafb5157d15664f3..6f9be772ffb8102028b6160ed4d34253d3c812c4 100644
--- a/chrome/browser/chromeos/login/kiosk_browsertest.cc
+++ b/chrome/browser/chromeos/login/kiosk_browsertest.cc
@@ -5,6 +5,7 @@
#include <memory>
#include <vector>
+#include "apps/test/app_window_waiter.h"
#include "ash/common/wallpaper/wallpaper_controller.h"
#include "ash/common/wallpaper/wallpaper_controller_observer.h"
#include "ash/common/wm_shell.h"
@@ -25,7 +26,6 @@
#include "chrome/browser/chromeos/file_manager/fake_disk_mount_manager.h"
#include "chrome/browser/chromeos/login/app_launch_controller.h"
#include "chrome/browser/chromeos/login/startup_utils.h"
-#include "chrome/browser/chromeos/login/test/app_window_waiter.h"
#include "chrome/browser/chromeos/login/test/oobe_base_test.h"
#include "chrome/browser/chromeos/login/test/oobe_screen_waiter.h"
#include "chrome/browser/chromeos/login/ui/login_display_host.h"
@@ -670,7 +670,7 @@ class KioskTest : public OobeBaseTest {
extensions::AppWindowRegistry* app_window_registry =
extensions::AppWindowRegistry::Get(app_profile);
extensions::AppWindow* window =
- AppWindowWaiter(app_window_registry, test_app_id_).Wait();
+ apps::AppWindowWaiter(app_window_registry, test_app_id_).Wait();
EXPECT_TRUE(window);
// Login screen should be gone or fading out.
@@ -860,7 +860,7 @@ IN_PROC_BROWSER_TEST_F(KioskTest, ZoomSupport) {
extensions::AppWindowRegistry* app_window_registry =
extensions::AppWindowRegistry::Get(app_profile);
extensions::AppWindow* window =
- AppWindowWaiter(app_window_registry, test_app_id()).Wait();
+ apps::AppWindowWaiter(app_window_registry, test_app_id()).Wait();
ASSERT_TRUE(window);
// Gets the original width of the app window.
@@ -2248,10 +2248,10 @@ IN_PROC_BROWSER_TEST_F(KioskEnterpriseTest, EnterpriseKioskApp) {
// Wait for the window to appear.
extensions::AppWindow* window =
- AppWindowWaiter(
- extensions::AppWindowRegistry::Get(
- ProfileManager::GetPrimaryUserProfile()),
- kTestEnterpriseKioskApp).Wait();
+ apps::AppWindowWaiter(extensions::AppWindowRegistry::Get(
+ ProfileManager::GetPrimaryUserProfile()),
+ kTestEnterpriseKioskApp)
+ .Wait();
ASSERT_TRUE(window);
// Check whether the app can retrieve an OAuth2 access token.

Powered by Google App Engine
This is Rietveld 408576698