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

Unified Diff: chrome/browser/ui/webui/bidi_checker_web_ui_test.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/ui/history_ui.cc ('k') | chrome/browser/ui/webui/browsing_history_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/bidi_checker_web_ui_test.cc
diff --git a/chrome/browser/ui/webui/bidi_checker_web_ui_test.cc b/chrome/browser/ui/webui/bidi_checker_web_ui_test.cc
index cf06ddfc929a0c67302ee8c67918582e6b7a68e1..acd5bd3da5de95bb962e5ca6f65fa505a23d4c7d 100644
--- a/chrome/browser/ui/webui/bidi_checker_web_ui_test.cc
+++ b/chrome/browser/ui/webui/bidi_checker_web_ui_test.cc
@@ -144,7 +144,7 @@ void DISABLED_WebUIBidiCheckerBrowserTestRTL::TearDownOnMainThread() {
// Tests
//==============================
-// chrome://settings/history
+// chrome://history
//==============================
static void SetupHistoryPageTest(Browser* browser,
@@ -165,13 +165,13 @@ IN_PROC_BROWSER_TEST_F(DISABLED_WebUIBidiCheckerBrowserTestLTR,
SetupHistoryPageTest(browser(),
"http://www.ynet.co.il",
"\xD7\x91\xD7\x93\xD7\x99\xD7\xA7\xD7\x94\x21");
- RunBidiCheckerOnPage(chrome::kChromeUIHistoryFrameURL);
+ RunBidiCheckerOnPage(chrome::kChromeUIHistoryURL);
}
IN_PROC_BROWSER_TEST_F(DISABLED_WebUIBidiCheckerBrowserTestRTL,
TestHistoryPage) {
SetupHistoryPageTest(browser(), "http://www.google.com", "Google");
- RunBidiCheckerOnPage(chrome::kChromeUIHistoryFrameURL);
+ RunBidiCheckerOnPage(chrome::kChromeUIHistoryURL);
}
//==============================
@@ -724,17 +724,3 @@ IN_PROC_BROWSER_TEST_F(DISABLED_WebUIBidiCheckerBrowserTestLTR, TestHelpFrame) {
IN_PROC_BROWSER_TEST_F(DISABLED_WebUIBidiCheckerBrowserTestRTL, TestHelpFrame) {
RunBidiCheckerOnPage(chrome::kChromeUIHelpFrameURL);
}
-
-//==============================
-// chrome://history-frame
-//==============================
-
-IN_PROC_BROWSER_TEST_F(DISABLED_WebUIBidiCheckerBrowserTestLTR,
- TestHistoryFrame) {
- RunBidiCheckerOnPage(chrome::kChromeUIHistoryFrameURL);
-}
-
-IN_PROC_BROWSER_TEST_F(DISABLED_WebUIBidiCheckerBrowserTestRTL,
- TestHistoryFrame) {
- RunBidiCheckerOnPage(chrome::kChromeUIHistoryFrameURL);
-}
« no previous file with comments | « chrome/browser/ui/history_ui.cc ('k') | chrome/browser/ui/webui/browsing_history_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698