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

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

Issue 2426683003: [MD settings] remove count of content items in content settings (Closed)
Patch Set: removed exception counter tests 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/resources/settings/site_settings/site_settings_category.html ('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 72cf64139f46e424ebe2270a9979b41a15dcccc5..cf8137cc7bd084630fe5d5d6ab8c1346fd4f5455 100644
--- a/chrome/test/data/webui/settings/site_list_tests.js
+++ b/chrome/test/data/webui/settings/site_list_tests.js
@@ -401,7 +401,6 @@ cr.define('site_list', function() {
assertEquals(
settings.PermissionValues.ALLOW, testElement.categorySubtype);
- assertEquals('Allow - 0', testElement.$.header.innerText.trim());
dschuyler 2016/10/18 19:10:58 I considered shortening these to just test that i
Dan Beam 2016/10/18 19:24:15 nah, just remove, IMO
assertFalse(testElement.$.category.hidden);
browserProxy.resetResolver('getExceptionList');
@@ -409,8 +408,6 @@ cr.define('site_list', function() {
return browserProxy.whenCalled('getExceptionList');
}).then(function(contentType) {
assertFalse(testElement.$.category.hidden);
- assertEquals('Exceptions - 0',
- testElement.$.header.innerText.trim());
});
});
@@ -430,7 +427,6 @@ cr.define('site_list', function() {
Polymer.dom.flush(); // Populates action menu.
openActionMenu(0);
assertMenu(['Block', 'Remove'], testElement);
- assertEquals('Allow - 2', testElement.$.header.innerText.trim());
// Site list should show, no matter what category default is set
// to.
@@ -440,8 +436,6 @@ cr.define('site_list', function() {
return browserProxy.whenCalled('getExceptionList');
}).then(function(contentType) {
assertFalse(testElement.$.category.hidden);
- assertEquals('Exceptions - 2',
- testElement.$.header.innerText.trim());
});
});
@@ -462,7 +456,6 @@ cr.define('site_list', function() {
Polymer.dom.flush(); // Populates action menu.
openActionMenu(0);
assertMenu(['Allow', 'Remove'], testElement);
- assertEquals('Block - 2', testElement.$.header.innerText.trim());
// Site list should only show when category default is enabled.
assertFalse(testElement.$.category.hidden);
@@ -491,8 +484,6 @@ cr.define('site_list', function() {
Polymer.dom.flush(); // Populates action menu.
openActionMenu(0);
assertMenu(['Allow', 'Block', 'Remove'], testElement);
- assertEquals('Clear on exit - 1',
- testElement.$.header.innerText.trim());
// Site list should show, no matter what category default is set
// to.
@@ -502,7 +493,6 @@ cr.define('site_list', function() {
return browserProxy.whenCalled('getExceptionList');
}).then(function(contentType) {
assertFalse(testElement.$.category.hidden);
- assertEquals('Clear on exit - 1', testElement.$.header.innerText);
});
});
@@ -523,8 +513,6 @@ cr.define('site_list', function() {
openActionMenu(0);
// 'Clear on exit' is visible as this is not an incognito item.
assertMenu(['Allow', 'Clear on exit', 'Remove'], testElement);
- assertEquals('Block - 1',
- testElement.$.header.innerText.trim());
// Select 'Remove from menu'.
var menuItems = getMenuItems(testElement.$.listContainer, 0);
@@ -560,8 +548,6 @@ cr.define('site_list', function() {
openActionMenu(0);
// 'Clear on exit' is hidden for incognito items.
assertMenu(['Block', 'Remove'], testElement);
- assertEquals('Allow - 2',
- testElement.$.header.innerText.trim());
// Select 'Remove' from menu on 'foo.com'.
openActionMenu(1);
« no previous file with comments | « chrome/browser/resources/settings/site_settings/site_settings_category.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698