MD Settings: Show all content settings in Site Details.

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

Index: chrome/test/data/webui/settings/site_details_tests.js
diff --git a/chrome/test/data/webui/settings/site_details_tests.js b/chrome/test/data/webui/settings/site_details_tests.js
index db9e3dec1ffafa0d7826f5181ecddedd7c2e34e4..ed4668cc8fa9b8188786301f27a91e787f47e7cd 100644
--- a/chrome/test/data/webui/settings/site_details_tests.js
+++ b/chrome/test/data/webui/settings/site_details_tests.js
@@ -115,34 +115,7 @@ suite('SiteDetails', function() {
     document.body.appendChild(testElement);
   });
 
-  test('empty state', function() {
-    var category = settings.ContentSettingsTypes.NOTIFICATIONS;
-    var site = {
-      origin: 'http://www.google.com',
-      displayName: 'http://www.google.com',
-      embeddingOrigin: '',
-    };
-    browserProxy.setPrefs(prefsEmpty);
-    testElement.category = category;
-    testElement.site = site
-
-    // expect usage to not be rendered
-    assertFalse(!!testElement.$$('#usage'));
-
-    // TODO(finnur): Check for the Permission heading hiding when no
-    // permissions are showing.
-
-    var msg = 'No category should be showing, height';
-    assertEquals(0, testElement.$.camera.offsetHeight, msg);
-    assertEquals(0, testElement.$.cookies.offsetHeight, msg);
-    assertEquals(0, testElement.$.geolocation.offsetHeight, msg);
-    assertEquals(0, testElement.$.javascript.offsetHeight, msg);
-    assertEquals(0, testElement.$.mic.offsetHeight, msg);
-    assertEquals(0, testElement.$.notification.offsetHeight, msg);
-    assertEquals(0, testElement.$.popups.offsetHeight, msg);
-  });
-
-  test('all categories visible', function() {
+  test('all categories', function() {
     var category = settings.ContentSettingsTypes.NOTIFICATIONS;
     var site = {
       origin: 'https://foo-allow.com:443',
@@ -155,13 +128,17 @@ suite('SiteDetails', function() {
     testElement.site = site;
 
     var msg = 'All categories should be showing';
-    assertFalse(testElement.$.camera.hidden, msg);
-    assertFalse(testElement.$.cookies.hidden, msg);
-    assertFalse(testElement.$.geolocation.hidden, msg);
-    assertFalse(testElement.$.javascript.hidden, msg);
-    assertFalse(testElement.$.mic.hidden, msg);
-    assertFalse(testElement.$.notification.hidden, msg);
-    assertFalse(testElement.$.popups.hidden, msg);
+    assertTrue(!!testElement.$.automaticDownloads);
+    assertTrue(!!testElement.$.backgroundSync);
+    assertTrue(!!testElement.$.camera);
+    assertTrue(!!testElement.$.cookies);
+    assertTrue(!!testElement.$.geolocation);
+    assertTrue(!!testElement.$.javascript);
+    assertTrue(!!testElement.$.mic);
+    assertTrue(!!testElement.$.notifications);
+    assertTrue(!!testElement.$.plugins);
+    assertTrue(!!testElement.$.popups);
+    assertTrue(!!testElement.$.unsandboxedPlugins);

[calamity, 2017/06/01 03:46:56]

I don't this tests what you want it to. It just checks the the element exists,
which will always be true. I think it's fine to remove this whole test.

[Patti Lor, 2017/06/01 04:52:01]

Done! Thanks, I was kinda unsure if it was really that useful any more.

   });
 
   test('usage heading shows on storage available', function() {