Better support for patterns.

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 69 matching lines @@
               embeddingOrigin: 'bar-allow.com',
               origin: 'bar-allow.com',
               setting: 'allow',
               source: 'preference',
             },
           ]
         }
       };
 
       /**
+       * An example pref with mixed origin and pattern.
+       * @type {SiteSettingsPref}
+       */
+      var prefsMixedOriginAndPattern = {
+        exceptions: {
+          media_stream_camera: [],
+          cookies: [],
+          fullscreen: [],
+          geolocation: [
+            {
+              origin: 'https://foo.com',
+              embeddingOrigin: '*',
+              setting: 'allow',
+              source: 'preference',
+            },
+          ],
+          javascript: [
+            {
+              origin: 'https://[*.]foo.com',
+              embeddingOrigin: '*',
+              setting: 'allow',
+              source: 'preference',
+            },
+          ],
+          images: [],
+          media_stream_mic: [],
+          notifications: [],
+          popups: [],
+        }
+      };
+
+      /**
        * An example pref with multiple categories and multiple allow/block
        * state.
        * @type {SiteSettingsPref}
        */
       var prefsVarious = {
         exceptions: {
           media_stream_camera: [],
           cookies: [],
           fullscreen: [],
           geolocation: [
@@ skipping 345 matching lines @@
                               prefsOneDisabled);
         return browserProxy.whenCalled('getExceptionList').then(
           function(contentType) {
             assertEquals(
                 settings.ContentSettingsTypes.GEOLOCATION, contentType);
 
             assertTrue(testElement.$.category.hidden);
         });
       });
 
-      test('All sites category', function(done) {
+      test('All sites category', function() {
         // Prefs: Multiple and overlapping sites.
         setupAllSitesCategory(prefsVarious);
 
-        browserProxy.whenCalled('getExceptionList').then(
+        return browserProxy.whenCalled('getExceptionList').then(
           function(contentType) {
-            testElement.async(function() {
+            // 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() {
               // 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.
-              // If this fails with 5 instead of the expected 3, then the
-              // de-duping of sites is not working for site_list.
-              assertEquals(3, testElement.sites.length);
+              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,
                   testElement.sites[2].origin);
               assertEquals(undefined, testElement.selectedOrigin);
 
               // Validate that the sites are shown in UI and can be selected.
               var firstItem = testElement.$.listContainer.items[1];
               var clickable = firstItem.querySelector('.flex paper-item-body');
               assertNotEquals(undefined, clickable);
               MockInteractions.tap(clickable);
               assertEquals(prefsVarious.exceptions.geolocation[0].origin,
                   testElement.selectedSite.origin);
+            });
+          });
+      });
 
-              done();
+      test('All sites mixed pattern and origin', function() {
+        // Prefs: One site, represented as origin and pattern.
+        setupAllSitesCategory(prefsMixedOriginAndPattern);
+
+        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() {
+              // 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);
+              // Validate that the sites are shown in UI and can be selected.
+              var firstItem = testElement.$.listContainer.items[0];
+              var clickable = firstItem.querySelector('.flex paper-item-body');
+              assertNotEquals(undefined, clickable);
+              MockInteractions.tap(clickable);
+              assertEquals(
+                  prefsMixedOriginAndPattern.exceptions.geolocation[0].origin,
+                  testElement.selectedSite.originForDisplay);
+            });
           });
-        });
       });
 
       test('Mixed schemes (present and absent)', function() {
         // Prefs: One item with scheme and one without.
         setupLocationCategory(settings.PermissionValues.ALLOW,
                               prefsMixedSchemes);
         return browserProxy.whenCalled('getExceptionList').then(
           function(contentType) {
             // No further checks needed. If this fails, it will hang the test.
         });
       });
     });
   }
   return {
     registerTests: registerTests,
   };
 });