MD Settings: Include settings for displaying Images in Site Details.

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

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /** @fileoverview Suite of tests for site-details. */
 suite('SiteDetails', function() {
   /**
    * A site list element created before each test.
    * @type {SiteDetails}
    */
@@ skipping 37 matching lines @@
         },
       ],
       geolocation: [
         {
           embeddingOrigin: 'https://foo-allow.com:443',
           origin: 'https://foo-allow.com:443',
           setting: 'allow',
           source: 'preference',
         },
       ],
+      images: [
+        {
+          embeddingOrigin: 'https://foo-allow.com:443',
+          origin: 'https://foo-allow.com:443',
+          setting: 'allow',
+          source: 'preference',
+        },
+      ],
       javascript: [
         {
           embeddingOrigin: 'https://foo-allow.com:443',
           origin: 'https://foo-allow.com:443',
           setting: 'allow',
           source: 'preference',
         },
       ],
       mic: [
         {
@@ skipping 84 matching lines @@
 
     browserProxy.setPrefs(prefs);
     testElement.category = category;
     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);

[tsergeant, 2017/06/01 04:02:24]

Should images be added to these test cases?

[Patti Lor, 2017/06/01 04:22:24]

Oops, yes. calamity@ said in a separate CL (that shows all the permissions all
the time) to delete this test though so I am leaving it as-is (see
https://codereview.chromium.org/2912253003/diff/60001/chrome/test/data/webui/...).

[tsergeant, 2017/06/01 04:26:21]

Fair enough

     assertFalse(testElement.$.notification.hidden, msg);
     assertFalse(testElement.$.popups.hidden, msg);
   });
 
   test('usage heading shows on storage available', function() {
     // Remove the current website-usage-private-api element.
     var parent = testElement.$.usageApi.parentNode;
     testElement.$.usageApi.remove();
 
     // Replace it with a mock version.
@@ skipping 14 matching lines @@
       displayName: 'https://foo-allow.com:443',
       embeddingOrigin: '',
     };
 
     Polymer.dom.flush();
 
     // expect usage to be rendered
     assertTrue(!!testElement.$$('#usage'));
   });
 });