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

Unified Diff: chrome/browser/resources/settings/site_settings/site_details_permission.js

Issue 2936003003: MD Settings: Set all content setting values in Site Details Javascript. (Closed)
Patch Set: Remove dependency on Page Info UI code and introduce new logic to retrieve content settings. 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/browser/resources/settings/site_settings/site_details_permission.js
diff --git a/chrome/browser/resources/settings/site_settings/site_details_permission.js b/chrome/browser/resources/settings/site_settings/site_details_permission.js
index 5cb99477cb0aab8b4f68381d2790a32098a018b2..4f8807b67074bc655389a899c7b5da306d5a010b 100644
--- a/chrome/browser/resources/settings/site_settings/site_details_permission.js
+++ b/chrome/browser/resources/settings/site_settings/site_details_permission.js
@@ -52,17 +52,7 @@ Polymer({
* @private
*/
siteChanged_: function(site) {
-
- this.browserProxy.getExceptionList(this.category).then(
- function(exceptionList) {
- for (var i = 0; i < exceptionList.length; ++i) {
- if (exceptionList[i].embeddingOrigin == site.embeddingOrigin &&
- this.sameOrigin_(exceptionList[i].origin, site.origin)) {
- this.$.permission.value = exceptionList[i].setting;
- break;
- }
- }
- }.bind(this));
+ this.$.permission.value = this.site.setting;
},
/**

Powered by Google App Engine
This is Rietveld 408576698