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

Unified Diff: chrome/browser/extensions/app_process_apitest.cc

Issue 10807047: Create content\public\test\test_utils.h to hold common test classes that are used by unit and brows… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 5 months 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/extensions/app_process_apitest.cc
===================================================================
--- chrome/browser/extensions/app_process_apitest.cc (revision 147577)
+++ chrome/browser/extensions/app_process_apitest.cc (working copy)
@@ -85,7 +85,7 @@
chrome::GetWebContentsAt(browser(), 1)->GetRenderProcessHost()->GetID()));
EXPECT_FALSE(chrome::GetWebContentsAt(browser(), 1)->GetWebUI());
- ui_test_utils::WindowedNotificationObserver tab_added_observer(
+ content::WindowedNotificationObserver tab_added_observer(
chrome::NOTIFICATION_TAB_ADDED,
content::NotificationService::AllSources());
chrome::NewTab(browser());
@@ -154,7 +154,7 @@
EXPECT_FALSE(chrome::GetWebContentsAt(browser(), 2)->GetWebUI());
LOG(INFO) << "Nav 2.";
- ui_test_utils::WindowedNotificationObserver tab_added_observer(
+ content::WindowedNotificationObserver tab_added_observer(
chrome::NOTIFICATION_TAB_ADDED,
content::NotificationService::AllSources());
chrome::NewTab(browser());
@@ -266,7 +266,7 @@
chrome::GetWebContentsAt(browser(), 1)->GetRenderProcessHost()->GetID()));
EXPECT_FALSE(chrome::GetWebContentsAt(browser(), 1)->GetWebUI());
- ui_test_utils::WindowedNotificationObserver tab_added_observer(
+ content::WindowedNotificationObserver tab_added_observer(
chrome::NOTIFICATION_TAB_ADDED,
content::NotificationService::AllSources());
chrome::NewTab(browser());
@@ -386,7 +386,7 @@
// Enable app and reload the page.
EnableExtension(app->id());
- ui_test_utils::WindowedNotificationObserver reload_observer(
+ content::WindowedNotificationObserver reload_observer(
content::NOTIFICATION_LOAD_STOP,
content::Source<NavigationController>(
&chrome::GetActiveWebContents(browser())->GetController()));
@@ -397,7 +397,7 @@
// Disable app and reload the page.
DisableExtension(app->id());
- ui_test_utils::WindowedNotificationObserver reload_observer2(
+ content::WindowedNotificationObserver reload_observer2(
content::NOTIFICATION_LOAD_STOP,
content::Source<NavigationController>(
&chrome::GetActiveWebContents(browser())->GetController()));
@@ -408,7 +408,7 @@
// Enable app and reload via JavaScript.
EnableExtension(app->id());
- ui_test_utils::WindowedNotificationObserver js_reload_observer(
+ content::WindowedNotificationObserver js_reload_observer(
content::NOTIFICATION_LOAD_STOP,
content::Source<NavigationController>(
&chrome::GetActiveWebContents(browser())->GetController()));
@@ -420,7 +420,7 @@
// Disable app and reload via JavaScript.
DisableExtension(app->id());
- ui_test_utils::WindowedNotificationObserver js_reload_observer2(
+ content::WindowedNotificationObserver js_reload_observer2(
content::NOTIFICATION_LOAD_STOP,
content::Source<NavigationController>(
&chrome::GetActiveWebContents(browser())->GetController()));
@@ -450,7 +450,7 @@
LoadExtension(test_data_dir_.AppendASCII("app_process"));
ASSERT_TRUE(app);
- ui_test_utils::WindowedNotificationObserver popup_observer(
+ content::WindowedNotificationObserver popup_observer(
content::NOTIFICATION_RENDER_VIEW_HOST_CREATED,
content::NotificationService::AllSources());
ui_test_utils::NavigateToURL(browser(),
@@ -528,7 +528,7 @@
LoadExtension(test_data_dir_.AppendASCII("app_process"));
ASSERT_TRUE(app);
- ui_test_utils::WindowedNotificationObserver popup_observer(
+ content::WindowedNotificationObserver popup_observer(
content::NOTIFICATION_RENDER_VIEW_HOST_CREATED,
content::NotificationService::AllSources());
ui_test_utils::NavigateToURL(browser(),
@@ -577,7 +577,7 @@
// Crash the tab and reload it, chrome.app.isInstalled should still be true.
ui_test_utils::CrashTab(chrome::GetActiveWebContents(browser()));
- ui_test_utils::WindowedNotificationObserver observer(
+ content::WindowedNotificationObserver observer(
content::NOTIFICATION_LOAD_STOP,
content::Source<NavigationController>(
&chrome::GetActiveWebContents(browser())->GetController()));
« no previous file with comments | « chrome/browser/extensions/app_background_page_apitest.cc ('k') | chrome/browser/extensions/content_script_apitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698