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

Unified Diff: chrome/test/data/webui/settings/cr_settings_browsertest.js

Issue 1666623006: MD Settings: Manage search engines 3/3, hooking up UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@manage_search_engines_handler
Patch Set: Addressing comments. Created 4 years, 10 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/test/data/webui/settings/cr_settings_browsertest.js
diff --git a/chrome/test/data/webui/settings/cr_settings_browsertest.js b/chrome/test/data/webui/settings/cr_settings_browsertest.js
index 6790547a46ce942fbbbcf6073391fe7cf4773692..9f57c7b6ac58cee140aa11acaa4a94a5c95730cf 100644
--- a/chrome/test/data/webui/settings/cr_settings_browsertest.js
+++ b/chrome/test/data/webui/settings/cr_settings_browsertest.js
@@ -36,6 +36,7 @@ CrSettingsBrowserTest.prototype = {
'prefs_test_cases.js',
'prefs_tests.js',
'reset_page_test.js',
+ 'search_engines_page_test.js',
'site_details_tests.js',
'site_details_permission_tests.js',
'site_list_tests.js',
@@ -69,9 +70,15 @@ TEST_F('CrSettingsBrowserTest', 'MAYBE_CrSettingsTest', function() {
mocha.run();
});
-
TEST_F('CrSettingsBrowserTest', 'ResetPage', function() {
settings_reset_page.registerDialogTests();
settings_reset_page.registerBannerTests();
mocha.run();
});
+
+TEST_F('CrSettingsBrowserTest', 'SearchEnginesPage', function() {
+ settings_search_engines_page.registerDialogTests();
+ settings_search_engines_page.registerEntryTests();
+ settings_search_engines_page.registerPageTests();
Dan Beam 2016/02/17 02:21:35 i don't know that you should be doing all of this
dpapad 2016/02/17 03:22:36 Any reasons why not? The tests share a bit of setu
+ mocha.run();
+});

Powered by Google App Engine
This is Rietveld 408576698