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

Unified Diff: chrome/browser/resources/options/content_settings_exceptions_area.js

Issue 528333002: Make it possible to revoke Web Notification exceptions in content settings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 | « no previous file | chrome/browser/ui/webui/options/content_settings_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/options/content_settings_exceptions_area.js
diff --git a/chrome/browser/resources/options/content_settings_exceptions_area.js b/chrome/browser/resources/options/content_settings_exceptions_area.js
index 2279c197c523490163be3eea58d60970ba39edba..e3e7b5bff3ac90d5ee99ff934557e411a8f5e6fe 100644
--- a/chrome/browser/resources/options/content_settings_exceptions_area.js
+++ b/chrome/browser/resources/options/content_settings_exceptions_area.js
@@ -579,13 +579,10 @@ cr.define('options.contentSettings', function() {
return;
var dataItem = listItem.dataItem;
- var args = [listItem.contentType];
- if (listItem.contentType == 'notifications')
- args.push(dataItem.origin, dataItem.setting);
- else
- args.push(listItem.mode, dataItem.origin, dataItem.embeddingOrigin);
-
- chrome.send('removeException', args);
+ chrome.send('removeException', [listItem.contentType,
+ listItem.mode,
+ dataItem.origin,
+ dataItem.embeddingOrigin]);
},
};
« no previous file with comments | « no previous file | chrome/browser/ui/webui/options/content_settings_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698