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

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

Issue 2769453002: MD Settings: Allow deleting read-only content setting exceptions. (Closed)
Patch Set: Created 3 years, 9 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
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 87d1b716a91780f21a9b15b2da219042347b3fb7..f6419132da2007eabeed7bea91e950e1a07fdfb5 100644
--- a/chrome/test/data/webui/settings/site_list_tests.js
+++ b/chrome/test/data/webui/settings/site_list_tests.js
@@ -388,7 +388,7 @@ cr.define('site_list', function() {
*/
function openActionMenu(index) {
var item = testElement.$.listContainer.children[index];
- var dots = item.querySelector('paper-icon-button');
+ var dots = item.querySelector('#actionMenuButton');
MockInteractions.tap(dots);
Polymer.dom.flush();
}
@@ -647,6 +647,47 @@ cr.define('site_list', function() {
});
});
+ test('reset button works for read-only content types', function() {
+ testElement.readOnlyList = true;
+ Polymer.dom.flush();
+
+ var contentType = settings.ContentSettingsTypes.GEOLOCATION;
+ var categorySubtype = settings.PermissionValues.ALLOW;
+ setUpCategory(contentType, categorySubtype, prefsOneEnabled);
+ return browserProxy.whenCalled('getExceptionList').then(
+ function(actualContentType) {
+ assertEquals(contentType, actualContentType);
+ assertEquals(categorySubtype, testElement.categorySubtype);
+
+ assertEquals(1, testElement.sites.length);
+ assertEquals(
+ prefsOneEnabled.exceptions.geolocation[0].origin,
+ testElement.sites[0].origin);
+
+ Polymer.dom.flush();
+
+ var item = testElement.$.listContainer.children[0];
+
+ // Assert action button is hidden.
+ var dots = item.querySelector('#actionMenuButton');
+ assertTrue(!!dots);
+ assertTrue(dots.hidden);
+
+ // Assert reset button is visible.
+ var resetButton = item.querySelector('#resetSite');
+ assertTrue(!!resetButton);
+ assertFalse(resetButton.hidden);
+
+ MockInteractions.tap(resetButton);
+ return browserProxy.whenCalled(
+ 'resetCategoryPermissionForOrigin');
+ }).then(function(args) {
+ assertEquals('https://foo-allow.com:443', args[0]);
+ assertEquals('https://foo-allow.com:443', args[1]);
+ assertEquals(contentType, args[2]);
+ });
+ });
+
test('edit action menu opens edit exception dialog', function() {
setUpCategory(
settings.ContentSettingsTypes.COOKIES,
@@ -787,7 +828,7 @@ cr.define('site_list', function() {
testElement.async(resolver.resolve);
return resolver.promise.then(function() {
var item = testElement.$.listContainer.children[0];
- var dots = item.querySelector('paper-icon-button');
+ var dots = item.querySelector('#actionMenuButton');
assertTrue(!!dots);
assertTrue(dots.hidden);
});

Powered by Google App Engine
This is Rietveld 408576698