MD Settings: Fix site data details regression and add test.

chrome/test/data/webui/settings/cr_settings_browsertest.js

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 c4620f4e9d63b63bf18affff27f8258a06fd5ae1..5bc54edc75dfbae942b955465d07d9edf03dbf85 100644
--- a/chrome/test/data/webui/settings/cr_settings_browsertest.js
+++ b/chrome/test/data/webui/settings/cr_settings_browsertest.js
@@ -663,7 +663,8 @@ CrSettingsSiteSettingsTest.prototype = {
     'test_site_settings_prefs_browser_proxy.js',
     'zoom_levels_tests.js',
     'usb_devices_tests.js',
-    'protocol_handlers_tests.js'
+    'protocol_handlers_tests.js',
+    'site_data_details_subpage_tests.js',
   ]),
 };
 
@@ -676,6 +677,7 @@ TEST_F('CrSettingsSiteSettingsTest', 'SiteSettings', function() {
   zoom_levels.registerTests();
   usb_devices.registerTests();
   protocol_handlers.registerTests();
+  site_data_details.registerTests();

[dpapad, 2017/02/08 00:04:10]

It is unfortunate that I have to use registerTests() here, but I could not find
a good way to avoid this. The new test I am adding is appending a "site=foo"
parameter to the URL, which affects the behavior of the site_details_tests.js
above. So essentially to preserve the old behavior of site_details_test.js, I
needed a way to either

 1 register the new test last, done via an explicit call to registerTests(), so
that the URL has not changed yet, by the time site_details_test.js runs, OR
 2 revert the URL change in the setup() of site_details_test.js.

I chose the 1st one.

[tommycli, 2017/02/08 19:28:40]

I probably would revert the URL change in the teardown of the test. Or -- if
that URL change is only applicable to that one test, do it in the test directly
(and revert in the test directly) instead of the setup/teardown methods.

I don't like to nitpick, but having this test be last seems to be fairly
fragile. If someone adds another test after this, it's going to mysteriously not
work without any obvious reason why.

I don't care if you leave it as a separate js file or combine it... that's up to
you.

[dpapad, 2017/02/08 22:01:09]

Done. Thanks for the help.

 
   mocha.run();
 });