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

Unified Diff: chrome/test/data/webui/settings/site_details_permission_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_permission_tests.js
diff --git a/chrome/test/data/webui/settings/site_details_permission_tests.js b/chrome/test/data/webui/settings/site_details_permission_tests.js
index 58a654d0a8565e663387f1fcfbe0e2c413e368bb..51bd52e7143ed42f0b10faa0be340d0a15ffc755 100644
--- a/chrome/test/data/webui/settings/site_details_permission_tests.js
+++ b/chrome/test/data/webui/settings/site_details_permission_tests.js
@@ -92,19 +92,15 @@ suite('SiteDetailsPermission', function() {
embeddingOrigin: '',
};
- return browserProxy.whenCalled('getExceptionList')
- .then(function() {
- assertFalse(testElement.$.details.hidden);
+ assertFalse(testElement.$.details.hidden);
- var header = testElement.$.details.querySelector('#permissionHeader');
- assertEquals(
- 'Camera', header.innerText.trim(),
- 'Widget should be labelled correctly');
+ var header = testElement.$.details.querySelector('#permissionHeader');
+ assertEquals(
+ 'Camera', header.innerText.trim(),
+ 'Widget should be labelled correctly');
- // Flip the permission and validate that prefs stay in sync.
- return validatePermissionFlipWorks(
- origin, settings.PermissionValues.ALLOW);
- })
+ // Flip the permission and validate that prefs stay in sync.
+ return validatePermissionFlipWorks(origin, settings.PermissionValues.ALLOW)
.then(function() {
return validatePermissionFlipWorks(
origin, settings.PermissionValues.BLOCK);
@@ -125,20 +121,17 @@ suite('SiteDetailsPermission', function() {
embeddingOrigin: '',
};
- return browserProxy.whenCalled('getExceptionList')
- .then(function() {
- assertFalse(testElement.$.details.hidden);
+ assertFalse(testElement.$.details.hidden);
- var header = testElement.$.details.querySelector('#permissionHeader');
- assertEquals(
- 'Cookies', header.innerText.trim(),
- 'Widget should be labelled correctly');
+ var header = testElement.$.details.querySelector('#permissionHeader');
+ assertEquals(
+ 'Cookies', header.innerText.trim(),
+ 'Widget should be labelled correctly');
- return validatePermissionFlipWorks(
- origin, settings.PermissionValues.SESSION_ONLY);
- })
+ // Flip the permission and validate that prefs stay in sync.
+ return validatePermissionFlipWorks(
+ origin, settings.PermissionValues.SESSION_ONLY)
.then(function() {
- // Flip the permission and validate that prefs stay in sync.
return validatePermissionFlipWorks(
origin, settings.PermissionValues.ALLOW);
})
« no previous file with comments | « chrome/browser/ui/webui/site_settings_helper.cc ('k') | chrome/test/data/webui/settings/site_details_tests.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698