summaryrefslogtreecommitdiffstats
path: root/application/palemoon/components/preferences/permissions.js
diff options
context:
space:
mode:
authorjanekptacijarabaci <janekptacijarabaci@seznam.cz>2018-04-26 12:26:15 +0200
committerjanekptacijarabaci <janekptacijarabaci@seznam.cz>2018-04-26 12:26:15 +0200
commit33377b3a9efd171f020fc84dc86d9c36b513887b (patch)
tree8e96e4faa9d75d47703870b54406cbf5688c5744 /application/palemoon/components/preferences/permissions.js
parent02f885d124989e70da3d2db93d8beab0aabd2586 (diff)
downloadUXP-33377b3a9efd171f020fc84dc86d9c36b513887b.tar
UXP-33377b3a9efd171f020fc84dc86d9c36b513887b.tar.gz
UXP-33377b3a9efd171f020fc84dc86d9c36b513887b.tar.lz
UXP-33377b3a9efd171f020fc84dc86d9c36b513887b.tar.xz
UXP-33377b3a9efd171f020fc84dc86d9c36b513887b.zip
Bug 1173523 - Part 5: Update the preferences permissions UI to use new API for nsIPermission (+ fix typos)
Diffstat (limited to 'application/palemoon/components/preferences/permissions.js')
-rw-r--r--application/palemoon/components/preferences/permissions.js78
1 files changed, 41 insertions, 37 deletions
diff --git a/application/palemoon/components/preferences/permissions.js b/application/palemoon/components/preferences/permissions.js
index 9fa2b84b9..f4b78022c 100644
--- a/application/palemoon/components/preferences/permissions.js
+++ b/application/palemoon/components/preferences/permissions.js
@@ -10,10 +10,10 @@ const nsICookiePermission = Components.interfaces.nsICookiePermission;
const NOTIFICATION_FLUSH_PERMISSIONS = "flush-pending-permissions";
-function Permission(host, rawHost, type, capability)
+function Permission(principal, type, capability)
{
- this.host = host;
- this.rawHost = rawHost;
+ this.principal = principal;
+ this.origin = principal.origin;
this.type = type;
this.capability = capability;
}
@@ -36,7 +36,7 @@ var gPermissionManager = {
getCellText: function (aRow, aColumn)
{
if (aColumn.id == "siteCol")
- return gPermissionManager._permissions[aRow].rawHost;
+ return gPermissionManager._permissions[aRow].origin;
else if (aColumn.id == "statusCol")
return gPermissionManager._permissions[aRow].capability;
return "";
@@ -83,10 +83,17 @@ var gPermissionManager = {
addPermission: function (aCapability)
{
var textbox = document.getElementById("url");
- var host = textbox.value.replace(/^\s*([-\w]*:\/+)?/, ""); // trim any leading space and scheme
+ var input_url = textbox.value.replace(/^\s*/, ""); // trim any leading space
+ let principal;
try {
- var uri = Services.io.newURI("http://" + host, null, null);
- host = uri.host;
+ // If the uri doesn't successfully parse, try adding a http:// and parsing again
+ let uri;
+ try {
+ let uri = Services.io.newURI(input_url, null, null);
+ } catch(ex) {
+ uri = Services.io.newURI("http://" + input_url, null, null);
+ }
+ principal = Services.scriptSecurityManager.getNoAppCodebasePrincipal(uri);
} catch(ex) {
var message = this._bundle.getString("invalidURI");
var title = this._bundle.getString("invalidURITitle");
@@ -97,11 +104,11 @@ var gPermissionManager = {
var capabilityString = this._getCapabilityString(aCapability);
// check whether the permission already exists, if not, add it
- let hostExists = false;
+ let permissionExists = false;
let capabilityExists = false;
for (var i = 0; i < this._permissions.length; ++i) {
- if (this._permissions[i].rawHost == host) {
- hostExists = true;
+ if (this._permissions[i].principal.equals(principal)) {
+ permissionExists = true;
capabilityExists = this._permissions[i].capability == capabilityString;
if (!capabilityExists) {
this._permissions[i].capability = capabilityString;
@@ -111,14 +118,14 @@ var gPermissionManager = {
}
- let permissionParams = {host: host, type: this._type, capability: aCapability};
- if (!hostExists) {
- this._permissionsToAdd.set(host, permissionParams);
+ let permissionParams = {principal: principal, type: this._type, capability: aCapability};
+ if (!permissionExists) {
+ this._permissionsToAdd.set(principal.origin, permissionParams);
this._addPermission(permissionParams);
}
else if (!capabilityExists) {
- this._permissionsToAdd.set(host, permissionParams);
- this._handleCapabilityChange();
+ this._permissionsToAdd.set(principal.origin, permissionParams);
+ this._handleCapabilityChange();
}
textbox.value = "";
@@ -133,15 +140,15 @@ var gPermissionManager = {
_removePermission: function(aPermission)
{
- this._removePermissionFromList(aPermission.host);
+ this._removePermissionFromList(aPermission.principal);
// If this permission was added during this session, let's remove
// it from the pending adds list to prevent calls to the
// permission manager.
- let isNewPermission = this._permissionsToAdd.delete(aPermission.host);
+ let isNewPermission = this._permissionsToAdd.delete(aPermission.principal.origin);
if (!isNewPermission) {
- this._permissionsToDelete.set(aPermission.host, aPermission);
+ this._permissionsToDelete.set(aPermission.principal.origin, aPermission);
}
},
@@ -278,7 +285,7 @@ var gPermissionManager = {
}
else if (aData == "changed") {
for (var i = 0; i < this._permissions.length; ++i) {
- if (this._permissions[i].host == permission.host) {
+ if (permission.matches(this._permissions[i].principal, true)) {
this._permissions[i].capability = this._getCapabilityString(permission.capability);
break;
}
@@ -366,13 +373,11 @@ var gPermissionManager = {
this.uninit();
for (let permissionParams of this._permissionsToAdd.values()) {
- let uri = Services.io.newURI("http://" + permissionParams.host, null, null);
- Services.perms.add(uri, permissionParams.type, permissionParams.capability);
+ Services.perms.addFromPrincipal(permissionParams.principal, permissionParams.type, permissionParams.capability);
}
for (let p of this._permissionsToDelete.values()) {
- let uri = Services.io.newURI("http://" + p.host, null, null);
- Services.perms.remove(uri, p.type);
+ Services.perms.removeFromPrincipal(p.principal, p.type);
}
window.close();
@@ -395,7 +400,7 @@ var gPermissionManager = {
// sort and display the table
this._tree.view = this._view;
- this.onPermissionSort("rawHost");
+ this.onPermissionSort("origin");
// disable "remove all" button if there are none
document.getElementById("removeAllPermissions").disabled = this._permissions.length == 0;
@@ -407,26 +412,19 @@ var gPermissionManager = {
(!this._manageCapability ||
(aPermission.capability == this._manageCapability))) {
- var host = aPermission.host;
+ var principal = aPermission.principal;
var capabilityString = this._getCapabilityString(aPermission.capability);
- var p = new Permission(host,
- (host.charAt(0) == ".") ? host.substring(1,host.length) : host,
+ var p = new Permission(principal,
aPermission.type,
capabilityString);
this._permissions.push(p);
}
},
- setHost: function (aHost)
- {
- document.getElementById("url").value = aHost;
- }
-};
-
- _removePermissionFromList: function (aHost)
+ _removePermissionFromList: function (aPrincipal)
{
for (let i = 0; i < this._permissions.length; ++i) {
- if (this._permissions[i].host == aHost) {
+ if (this._permissions[i].principal.equals(aPrincipal)) {
this._permissions.splice(i, 1);
this._view._rowCount--;
this._tree.treeBoxObject.rowCountChanged(this._view.rowCount - 1, -1);
@@ -436,9 +434,15 @@ var gPermissionManager = {
}
},
-function setHost(aHost)
+ setOrigin: function (aOrigin)
+ {
+ document.getElementById("url").value = aOrigin;
+ }
+};
+
+function setOrigin(aOrigin)
{
- gPermissionManager.setHost(aHost);
+ gPermissionManager.setOrigin(aOrigin);
}
function initWithParams(aParams)