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

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

Issue 2936003003: MD Settings: Set all content setting values in Site Details Javascript. (Closed)
Patch Set: Review comments. Created 3 years, 6 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_details_tests.js
diff --git a/chrome/test/data/webui/settings/site_details_tests.js b/chrome/test/data/webui/settings/site_details_tests.js
index 95517a16dace599e0e6d8ca76acdd6e03e506193..b19508208c231c47ce5f2a061396fe5046be3585 100644
--- a/chrome/test/data/webui/settings/site_details_tests.js
+++ b/chrome/test/data/webui/settings/site_details_tests.js
@@ -36,7 +36,7 @@ suite('SiteDetails', function() {
embeddingOrigin: 'https://foo-allow.com:443',
origin: 'https://foo-allow.com:443',
setting: 'allow',
- source: 'preference',
+ source: 'extension',
},
],
cookies: [
@@ -51,8 +51,8 @@ suite('SiteDetails', function() {
{
embeddingOrigin: 'https://foo-allow.com:443',
origin: 'https://foo-allow.com:443',
- setting: 'allow',
- source: 'preference',
+ setting: 'block',
+ source: 'policy',
},
],
images: [
@@ -141,15 +141,28 @@ suite('SiteDetails', function() {
Polymer.dom(parent).appendChild(api);
browserProxy.setPrefs(prefs);
- testElement.site = {
- origin: 'https://foo-allow.com:443',
- displayName: 'https://foo-allow.com:443',
- embeddingOrigin: '',
- };
+ testElement.origin = 'https://foo-allow.com:443';
Polymer.dom.flush();
- // expect usage to be rendered
+ // Expect usage to be rendered.
assertTrue(!!testElement.$$('#usage'));
});
+
+ test('correct pref settings are shown', function() {
+ browserProxy.setPrefs(prefs);
+ testElement.origin = 'https://foo-allow.com:443';
+
+ return browserProxy.whenCalled('getOriginPermissions').then(function() {
+ testElement.root.querySelectorAll('site-details-permission')
+ .forEach(function(siteDetailsPermission) {
+ // Verify settings match the values specified in |prefs|.
+ var setting = 'allow';
+ if (siteDetailsPermission.site.category == 'location')
+ setting = 'block';
+ assertEquals(setting, siteDetailsPermission.site.setting);
+ assertEquals(setting, siteDetailsPermission.$.permission.value);
+ });
+ });
+ });
});

Powered by Google App Engine
This is Rietveld 408576698