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

Unified Diff: chrome/browser/browser_about_handler_unittest.cc

Issue 2830983005: Remove old webui History page on desktop and mobile (Closed)
Patch Set: merge Created 3 years, 7 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
« no previous file with comments | « chrome/browser/browser_about_handler.cc ('k') | chrome/browser/browser_resources.grd » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browser_about_handler_unittest.cc
diff --git a/chrome/browser/browser_about_handler_unittest.cc b/chrome/browser/browser_about_handler_unittest.cc
index 321fd720b95d79b4e56ad7df47fbf9b951d9c2fe..03e2ebb8d37e323d807fb69944d87062350fe110 100644
--- a/chrome/browser/browser_about_handler_unittest.cc
+++ b/chrome/browser/browser_about_handler_unittest.cc
@@ -130,6 +130,19 @@ TEST_F(BrowserAboutHandlerTest, WillHandleBrowserAboutURLForMDSettings) {
TestWillHandleBrowserAboutURL(test_cases);
}
+TEST_F(BrowserAboutHandlerTest, WillHandleBrowserAboutURLForHistory) {
+ TestWillHandleBrowserAboutURL(std::vector<AboutURLTestCase>({
+ {GURL("about:history"), GURL("chrome://history/")},
+ {GURL("about:history-frame"), GURL("chrome://history/")},
+ {GURL("chrome://history"), GURL("chrome://history/")},
+ {GURL("chrome://history-frame"), GURL("chrome://history/")},
+ {GURL("chrome://history/"), GURL("chrome://history/")},
+ {GURL("chrome://history-frame/"), GURL("chrome://history/")},
+ {GURL("chrome://history/?q=foo"), GURL("chrome://history/?q=foo")},
+ {GURL("chrome://history-frame/?q=foo"), GURL("chrome://history/?q=foo")},
+ }));
+}
+
// Ensure that minor BrowserAboutHandler fixup to a URL does not cause us to
// keep a separate virtual URL, which would not be updated on redirects.
// See https://crbug.com/449829.
« no previous file with comments | « chrome/browser/browser_about_handler.cc ('k') | chrome/browser/browser_resources.grd » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698