[MD settings] content settings add site button; layout fixes

chrome/test/data/webui/settings/site_list_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-list. */
 cr.define('site_list', function() {
   function registerTests() {
     suite('SiteList', function() {
       /**
        * A site list element created before each test.
@@ skipping 582 matching lines @@
       });
 
       test('Block list open when Allow list is empty', function() {
         // Prefs: One item in Block list, nothing in Allow list.
         setupCategory(settings.ContentSettingsTypes.GEOLOCATION,
             settings.PermissionValues.BLOCK, prefsOneDisabled);
         return browserProxy.whenCalled('getExceptionList').then(
             function(contentType) {
               assertEquals(
                   settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
               assertFalse(testElement.$.category.hidden);
-              assertTrue(testElement.$.category.opened);
             }).then(function() {
               assertNotEquals(0, testElement.$.listContainer.offsetHeight);
             });
       });
 
       test('Block list closed when Allow list is not empty', function() {
         // Prefs: Items in both Block and Allow list.
         setupCategory(settings.ContentSettingsTypes.GEOLOCATION,
             settings.PermissionValues.BLOCK, prefs);
         return browserProxy.whenCalled('getExceptionList').then(
             function(contentType) {
               assertEquals(
                   settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
               assertFalse(testElement.$.category.hidden);
-              assertFalse(testElement.$.category.opened);
               assertEquals(0, testElement.$.listContainer.offsetHeight);
             });
       });
 
       test('Allow list is always open (Block list empty)', function() {
         // Prefs: One item in Allow list, nothing in Block list.
         setupCategory(settings.ContentSettingsTypes.GEOLOCATION,
             settings.PermissionValues.ALLOW, prefsOneEnabled);
         return browserProxy.whenCalled('getExceptionList').then(
             function(contentType) {
               assertEquals(
                   settings.ContentSettingsTypes.GEOLOCATION, contentType);
 
               assertFalse(testElement.$.category.hidden);
-              assertTrue(testElement.$.category.opened);
             }).then(function() {
               assertNotEquals(0, testElement.$.listContainer.offsetHeight);
             });
       });
 
       test('Allow list is always open (Block list non-empty)', function() {
         // Prefs: Items in both Block and Allow list.
         setupCategory(settings.ContentSettingsTypes.GEOLOCATION,
             settings.PermissionValues.ALLOW, prefs);
         return browserProxy.whenCalled('getExceptionList').then(
             function(contentType) {
               assertEquals(
                   settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
               assertFalse(testElement.$.category.hidden);
-              assertTrue(testElement.$.category.opened);
             }).then(function() {
               assertNotEquals(0, testElement.$.listContainer.offsetHeight);
             });
       });
 
       test('Block list not hidden when empty', function() {
         // Prefs: One item in Allow list, nothing in Block list.
         setupCategory(settings.ContentSettingsTypes.GEOLOCATION,
             settings.PermissionValues.BLOCK, prefsOneEnabled);
         return browserProxy.whenCalled('getExceptionList').then(
             function(contentType) {
               assertEquals(
                   settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
               assertFalse(testElement.$.category.hidden);
             });
       });
 
       test('Allow list not hidden when empty', function() {
         // Prefs: One item in Block list, nothing in Allow list.
         setupCategory(settings.ContentSettingsTypes.GEOLOCATION,
             settings.PermissionValues.ALLOW, prefsOneDisabled);
         return browserProxy.whenCalled('getExceptionList').then(
             function(contentType) {
               assertEquals(
                   settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
               assertFalse(testElement.$.category.hidden);
             });
       });
 
       test('All sites category', function() {
         // Prefs: Multiple and overlapping sites.
         setupCategory(settings.ALL_SITES, '', prefsVarious);
 
         return browserProxy.whenCalled('getExceptionList').then(
             function(contentType) {
               // Use resolver to ensure asserts bubble up to the framework with
               // meaningful errors.
               var resolver = new PromiseResolver();
               testElement.async(resolver.resolve);
               return resolver.promise.then(function() {
                 testElement.enableSiteSettings_ = true;
                 // All Sites calls getExceptionList for all categories, starting
                 // with Cookies.
                 assertEquals(
                     settings.ContentSettingsTypes.COOKIES, contentType);
 
                 // Required for firstItem to be found below.
                 Polymer.dom.flush();
 
-                assertTrue(testElement.$.category.opened);
                 assertFalse(testElement.$.category.hidden);
                 // Validate that the sites gets populated from pre-canned prefs.
                 assertEquals(3, testElement.sites.length,
                     'If this fails with 5 instead of the expected 3, then ' +
                     'the de-duping of sites is not working for site_list');
                 assertEquals(prefsVarious.exceptions.geolocation[1].origin,
                     testElement.sites[0].origin);
                 assertEquals(prefsVarious.exceptions.geolocation[0].origin,
                     testElement.sites[1].origin);
                 assertEquals(prefsVarious.exceptions.notifications[0].origin,
@@ skipping 24 matching lines @@
               return resolver.promise.then(function() {
                 testElement.enableSiteSettings_ = true;
                 // All Sites calls getExceptionList for all categories, starting
                 // with Cookies.
                 assertEquals(
                     settings.ContentSettingsTypes.COOKIES, contentType);
 
                 // Required for firstItem to be found below.
                 Polymer.dom.flush();
 
-                assertTrue(testElement.$.category.opened);
                 assertFalse(testElement.$.category.hidden);
                 // Validate that the sites gets populated from pre-canned prefs.
                 assertEquals(1, testElement.sites.length,
                     'If this fails with 2 instead of the expected 1, then ' +
                     'the de-duping of sites is not working for site_list');
                 assertEquals(
                     prefsMixedOriginAndPattern.exceptions.geolocation[0].origin,
                     testElement.sites[0].originForDisplay);
 
                 assertEquals(undefined, testElement.selectedOrigin);
@@ skipping 54 matching lines @@
           assertEquals(settings.ContentSettingsTypes.JAVASCRIPT, args[2]);
           assertEquals('allow', args[3]);
         });
       });
     });
   }
   return {
     registerTests: registerTests,
   };
 });