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

Unified Diff: chrome/browser/chromeos/offline/offline_load_page_unittest.cc

Issue 19869003: ChromeOS changes for GetURL refactor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixing failing tests. Created 7 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/chromeos/offline/offline_load_page_unittest.cc
diff --git a/chrome/browser/chromeos/offline/offline_load_page_unittest.cc b/chrome/browser/chromeos/offline/offline_load_page_unittest.cc
index b0b4bec8213f169ba8c1394cd4e2de38377ef796..6265d831c29be0c3d2b146354d1d14b995a062b3 100644
--- a/chrome/browser/chromeos/offline/offline_load_page_unittest.cc
+++ b/chrome/browser/chromeos/offline/offline_load_page_unittest.cc
@@ -114,7 +114,7 @@ TEST_F(OfflineLoadPageTest, OfflinePageProceed) {
EXPECT_EQ(OK, user_response());
// The URL remains to be URL2.
- EXPECT_EQ(kURL2, web_contents()->GetURL().spec());
+ EXPECT_EQ(kURL2, web_contents()->GetVisibleURL().spec());
// Commit navigation and the interstitial page is gone.
Navigate(kURL2, 2);
@@ -143,7 +143,7 @@ TEST_F(OfflineLoadPageTest, OfflinePageDontProceed) {
// We did not proceed, the pending entry should be gone.
EXPECT_FALSE(controller().GetPendingEntry());
// the URL is set back to kURL1.
- EXPECT_EQ(kURL1, web_contents()->GetURL().spec());
+ EXPECT_EQ(kURL1, web_contents()->GetLastCommittedURL().spec());
}
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698