Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4902)

Unified Diff: chrome/test/data/webui/settings/site_list_tests.js

Issue 2429973003: [MD settings] content settings add site button; layout fixes (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 cf8137cc7bd084630fe5d5d6ab8c1346fd4f5455..13e6d8f3d46d7eb5bd709018259f035eee7db78a 100644
--- a/chrome/test/data/webui/settings/site_list_tests.js
+++ b/chrome/test/data/webui/settings/site_list_tests.js
@@ -600,9 +600,7 @@ cr.define('site_list', function() {
function(contentType) {
assertEquals(
settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
assertFalse(testElement.$.category.hidden);
- assertTrue(testElement.$.category.opened);
}).then(function() {
assertNotEquals(0, testElement.$.listContainer.offsetHeight);
});
@@ -616,9 +614,7 @@ cr.define('site_list', function() {
function(contentType) {
assertEquals(
settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
assertFalse(testElement.$.category.hidden);
- assertFalse(testElement.$.category.opened);
assertEquals(0, testElement.$.listContainer.offsetHeight);
});
});
@@ -633,7 +629,6 @@ cr.define('site_list', function() {
settings.ContentSettingsTypes.GEOLOCATION, contentType);
assertFalse(testElement.$.category.hidden);
- assertTrue(testElement.$.category.opened);
}).then(function() {
assertNotEquals(0, testElement.$.listContainer.offsetHeight);
});
@@ -647,9 +642,7 @@ cr.define('site_list', function() {
function(contentType) {
assertEquals(
settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
assertFalse(testElement.$.category.hidden);
- assertTrue(testElement.$.category.opened);
}).then(function() {
assertNotEquals(0, testElement.$.listContainer.offsetHeight);
});
@@ -663,7 +656,6 @@ cr.define('site_list', function() {
function(contentType) {
assertEquals(
settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
assertFalse(testElement.$.category.hidden);
});
});
@@ -676,7 +668,6 @@ cr.define('site_list', function() {
function(contentType) {
assertEquals(
settings.ContentSettingsTypes.GEOLOCATION, contentType);
-
assertFalse(testElement.$.category.hidden);
});
});
@@ -701,7 +692,6 @@ cr.define('site_list', function() {
// 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,
@@ -746,7 +736,6 @@ cr.define('site_list', function() {
// 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,
« no previous file with comments | « chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698