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

Unified Diff: content/browser/renderer_host/render_view_host_manager_browsertest.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: content/browser/renderer_host/render_view_host_manager_browsertest.cc
===================================================================
--- content/browser/renderer_host/render_view_host_manager_browsertest.cc (revision 147577)
+++ content/browser/renderer_host/render_view_host_manager_browsertest.cc (working copy)
@@ -401,7 +401,7 @@
// Clicking the original link in the first tab should cause us to swap back.
chrome::ActivateTabAt(browser(), 0, true);
- ui_test_utils::WindowedNotificationObserver navigation_observer(
+ content::WindowedNotificationObserver navigation_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&new_contents->GetController()));
@@ -425,7 +425,7 @@
EXPECT_EQ(new_site_instance,
chrome::GetActiveWebContents(browser())->GetSiteInstance());
chrome::ActivateTabAt(browser(), 0, true);
- ui_test_utils::WindowedNotificationObserver close_observer(
+ content::WindowedNotificationObserver close_observer(
content::NOTIFICATION_WEB_CONTENTS_DESTROYED,
content::Source<content::WebContents>(new_contents));
EXPECT_TRUE(ui_test_utils::ExecuteJavaScriptAndExtractBool(
@@ -539,7 +539,7 @@
// 3) Post a message from the foo window to the opener. The opener will
// reply, causing the foo window to update its own title.
- ui_test_utils::WindowedNotificationObserver title_observer(
+ content::WindowedNotificationObserver title_observer(
content::NOTIFICATION_WEB_CONTENTS_TITLE_UPDATED,
content::Source<content::WebContents>(foo_contents));
EXPECT_TRUE(ui_test_utils::ExecuteJavaScriptAndExtractBool(
@@ -567,7 +567,7 @@
// 4) Now post a message from the _blank window to the foo window. The
// foo window will update its title and will not reply.
- ui_test_utils::WindowedNotificationObserver title_observer2(
+ content::WindowedNotificationObserver title_observer2(
content::NOTIFICATION_WEB_CONTENTS_TITLE_UPDATED,
content::Source<content::WebContents>(foo_contents));
EXPECT_TRUE(ui_test_utils::ExecuteJavaScriptAndExtractBool(
@@ -646,7 +646,7 @@
// The opened tab should be able to navigate the opener back to its process.
chrome::ActivateTabAt(browser(), 1, true);
- ui_test_utils::WindowedNotificationObserver navigation_observer(
+ content::WindowedNotificationObserver navigation_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&orig_contents->GetController()));
@@ -735,7 +735,7 @@
// Navigate the first tab to a different site as well. The original process
// should exit, since all of its views are now swapped out.
chrome::ActivateTabAt(browser(), 0, true);
- ui_test_utils::WindowedNotificationObserver exit_observer(
+ content::WindowedNotificationObserver exit_observer(
content::NOTIFICATION_RENDERER_PROCESS_TERMINATED,
content::Source<content::RenderProcessHost>(orig_process));
ui_test_utils::NavigateToURL(browser(),
@@ -865,7 +865,7 @@
// Go back three times to first site.
{
- ui_test_utils::WindowedNotificationObserver back_nav_load_observer(
+ content::WindowedNotificationObserver back_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&contents->GetController()));
@@ -873,7 +873,7 @@
back_nav_load_observer.Wait();
}
{
- ui_test_utils::WindowedNotificationObserver back_nav_load_observer(
+ content::WindowedNotificationObserver back_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&contents->GetController()));
@@ -881,7 +881,7 @@
back_nav_load_observer.Wait();
}
{
- ui_test_utils::WindowedNotificationObserver back_nav_load_observer(
+ content::WindowedNotificationObserver back_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&contents->GetController()));
@@ -891,7 +891,7 @@
// Now go forward twice to B2. Shouldn't be left spinning.
{
- ui_test_utils::WindowedNotificationObserver forward_nav_load_observer(
+ content::WindowedNotificationObserver forward_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&contents->GetController()));
@@ -899,7 +899,7 @@
forward_nav_load_observer.Wait();
}
{
- ui_test_utils::WindowedNotificationObserver forward_nav_load_observer(
+ content::WindowedNotificationObserver forward_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&contents->GetController()));
@@ -909,7 +909,7 @@
// Go back twice to first site.
{
- ui_test_utils::WindowedNotificationObserver back_nav_load_observer(
+ content::WindowedNotificationObserver back_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&contents->GetController()));
@@ -917,7 +917,7 @@
back_nav_load_observer.Wait();
}
{
- ui_test_utils::WindowedNotificationObserver back_nav_load_observer(
+ content::WindowedNotificationObserver back_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&contents->GetController()));
@@ -927,7 +927,7 @@
// Now go forward directly to B3. Shouldn't be left spinning.
{
- ui_test_utils::WindowedNotificationObserver forward_nav_load_observer(
+ content::WindowedNotificationObserver forward_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&contents->GetController()));
@@ -1003,7 +1003,7 @@
// Going back should make the previously swapped-out view to become visible
// again.
{
- ui_test_utils::WindowedNotificationObserver back_nav_load_observer(
+ content::WindowedNotificationObserver back_nav_load_observer(
content::NOTIFICATION_NAV_ENTRY_COMMITTED,
content::Source<content::NavigationController>(
&chrome::GetActiveWebContents(browser())->GetController()));

Powered by Google App Engine
This is Rietveld 408576698