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

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

Index: chrome/test/data/webui/settings/site_list_tests.js
diff --git a/chrome/test/data/webui/settings/site_list_tests.js b/chrome/test/data/webui/settings/site_list_tests.js
index 7b13e5c464720977960abf966dfde55567446f18..3ccf52f6a4731d5c2498ca6131521ff68649af8a 100644
--- a/chrome/test/data/webui/settings/site_list_tests.js
+++ b/chrome/test/data/webui/settings/site_list_tests.js
@@ -344,6 +344,9 @@ cr.define('site_list', function() {
 
       teardown(function() {
         closeActionMenu();
+        // The code being tested changes the URL. Reset so that URL state is not
+        // leaked across tests.
+        settings.navigateTo(settings.Route.BASIC);

[tommycli, 2017/02/08 22:04:17]

well technically it's the internal route variable that matters, not the actual
URL...

[dpapad, 2017/02/08 22:16:42]

I see. In that case, how about calling resetRouteForTesting() instead (see new
patch)? (I had added that method in a previous CL, necessary for when tests run
in vulcanized mode).

       });
 
       /**