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

Unified Diff: content/browser/web_contents/navigation_controller_impl_unittest.cc

Issue 11275062: Move content\browser\web_contents to content namespace. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 2 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/web_contents/navigation_controller_impl_unittest.cc
===================================================================
--- content/browser/web_contents/navigation_controller_impl_unittest.cc (revision 164670)
+++ content/browser/web_contents/navigation_controller_impl_unittest.cc (working copy)
@@ -2914,17 +2914,17 @@
const GURL kPageWithFavicon("http://withfavicon.html");
const GURL kPageWithoutFavicon("http://withoutfavicon.html");
const GURL kIconURL("http://withfavicon.ico");
- const gfx::Image kDefaultFavicon = content::FaviconStatus().image;
+ const gfx::Image kDefaultFavicon = FaviconStatus().image;
NavigationControllerImpl& controller = controller_impl();
- content::TestNotificationTracker notifications;
+ TestNotificationTracker notifications;
RegisterForAllNavNotifications(&notifications, &controller);
test_rvh()->SendNavigate(0, kPageWithFavicon);
EXPECT_TRUE(notifications.Check1AndReset(
- content::NOTIFICATION_NAV_ENTRY_COMMITTED));
+ NOTIFICATION_NAV_ENTRY_COMMITTED));
- content::NavigationEntry* entry = controller.GetLastCommittedEntry();
+ NavigationEntry* entry = controller.GetLastCommittedEntry();
EXPECT_TRUE(entry);
EXPECT_EQ(kPageWithFavicon, entry->GetURL());
@@ -2933,7 +2933,7 @@
bitmap.setConfig(SkBitmap::kARGB_8888_Config, 1, 1);
bitmap.allocPixels();
- content::FaviconStatus& favicon_status = entry->GetFavicon();
+ FaviconStatus& favicon_status = entry->GetFavicon();
favicon_status.image = gfx::Image(bitmap);
favicon_status.url = kIconURL;
favicon_status.valid = true;
@@ -2942,9 +2942,9 @@
test_rvh()->SendNavigateWithTransition(
0, // same page ID.
kPageWithoutFavicon,
- content::PAGE_TRANSITION_CLIENT_REDIRECT);
+ PAGE_TRANSITION_CLIENT_REDIRECT);
EXPECT_TRUE(notifications.Check1AndReset(
- content::NOTIFICATION_NAV_ENTRY_COMMITTED));
+ NOTIFICATION_NAV_ENTRY_COMMITTED));
entry = controller.GetLastCommittedEntry();
EXPECT_TRUE(entry);
@@ -2961,21 +2961,21 @@
const GURL kIconURL("http://www.a.com/1/favicon.ico");
NavigationControllerImpl& controller = controller_impl();
- content::TestNotificationTracker notifications;
+ TestNotificationTracker notifications;
RegisterForAllNavNotifications(&notifications, &controller);
test_rvh()->SendNavigate(0, kUrl1);
EXPECT_TRUE(notifications.Check1AndReset(
- content::NOTIFICATION_NAV_ENTRY_COMMITTED));
+ NOTIFICATION_NAV_ENTRY_COMMITTED));
tfarina 2012/10/29 19:37:19 will fit above.
jam 2012/10/29 20:01:51 Done.
// Simulate Chromium having set the favicon for |kUrl1|.
SkBitmap favicon_bitmap;
favicon_bitmap.setConfig(SkBitmap::kARGB_8888_Config, 1, 1);
favicon_bitmap.allocPixels();
- content::NavigationEntry* entry = controller.GetLastCommittedEntry();
+ NavigationEntry* entry = controller.GetLastCommittedEntry();
EXPECT_TRUE(entry);
- content::FaviconStatus& favicon_status = entry->GetFavicon();
+ FaviconStatus& favicon_status = entry->GetFavicon();
favicon_status.image = gfx::Image(favicon_bitmap);
favicon_status.url = kIconURL;
favicon_status.valid = true;
@@ -2983,13 +2983,13 @@
// Navigate to another page and go back to the original page.
test_rvh()->SendNavigate(1, kUrl2);
EXPECT_TRUE(notifications.Check1AndReset(
- content::NOTIFICATION_NAV_ENTRY_COMMITTED));
+ NOTIFICATION_NAV_ENTRY_COMMITTED));
tfarina 2012/10/29 19:37:19 will fit above.
jam 2012/10/29 20:01:51 Done.
test_rvh()->SendNavigateWithTransition(
0,
kUrl1,
- content::PAGE_TRANSITION_FORWARD_BACK);
+ PAGE_TRANSITION_FORWARD_BACK);
EXPECT_TRUE(notifications.Check1AndReset(
- content::NOTIFICATION_NAV_ENTRY_COMMITTED));
+ NOTIFICATION_NAV_ENTRY_COMMITTED));
tfarina 2012/10/29 19:37:19 will fit above.
jam 2012/10/29 20:01:51 Done.
// Verify that the favicon for the page at |kUrl1| was not cleared.
gfx::Image favicon_after_back;

Powered by Google App Engine
This is Rietveld 408576698