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

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

Issue 2855943004: [MD settings] split protocol handler tests from site tests (Closed)
Patch Set: Created 3 years, 8 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 b4cb8b9ec71e0549205c03d0ae5c033d059322e4..376938e9c2bbe8b5f2d31ed5ae9ce9c0b7437368 100644
--- a/chrome/test/data/webui/settings/cr_settings_browsertest.js
+++ b/chrome/test/data/webui/settings/cr_settings_browsertest.js
@@ -805,7 +805,6 @@ CrSettingsSiteSettingsTest.prototype = {
'test_site_settings_prefs_browser_proxy.js',
'zoom_levels_tests.js',
'usb_devices_tests.js',
- 'protocol_handlers_tests.js',
'site_data_details_subpage_tests.js',
]),
};
@@ -818,7 +817,6 @@ TEST_F('CrSettingsSiteSettingsTest', 'SiteSettings', function() {
site_list.registerTests();
zoom_levels.registerTests();
usb_devices.registerTests();
- protocol_handlers.registerTests();
mocha.run();
});
@@ -827,6 +825,30 @@ TEST_F('CrSettingsSiteSettingsTest', 'SiteSettings', function() {
* @constructor
* @extends {CrSettingsBrowserTest}
*/
+function CrSettingsProtocolHandlersTest() {}
+
+CrSettingsProtocolHandlersTest.prototype = {
+ __proto__: CrSettingsBrowserTest.prototype,
+
+ /** @override */
+ browsePreload: 'chrome://md-settings/privacy_page/privacy_page.html',
+
+ /** @override */
+ extraLibraries: CrSettingsBrowserTest.prototype.extraLibraries.concat([
+ 'test_browser_proxy.js',
+ 'test_site_settings_prefs_browser_proxy.js',
+ 'protocol_handlers_tests.js',
+ ]),
+};
+
+TEST_F('CrSettingsProtocolHandlersTest', 'ProtocolHandlers', function() {
+ mocha.run();
+});
+
+/**
+ * @constructor
+ * @extends {CrSettingsBrowserTest}
+ */
function CrSettingsSiteDataTest() {}
CrSettingsSiteDataTest.prototype = {

Powered by Google App Engine
This is Rietveld 408576698