commit c6eb1efb7d3ca6a9b5e19f6ec352ee1b129efbd3 Author: Andrei Lazar andrei.a.lazar@softvision.ro Date: Wed Jul 24 16:00:32 2019 +0000

Bug 1568521 Tracking protection doorhanger's Enable/Disable states don't work r=VladBaicu

Now using lastURI instead of principal and now we are using the right functions in order to test permissions.

Differential Revision: https://phabricator.services.mozilla.com/D39190

--HG-- extra : moz-landing-system : lando --- mobile/android/chrome/content/PermissionsHelper.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/mobile/android/chrome/content/PermissionsHelper.js b/mobile/android/chrome/content/PermissionsHelper.js index b25920872635..39556e756c84 100644 --- a/mobile/android/chrome/content/PermissionsHelper.js +++ b/mobile/android/chrome/content/PermissionsHelper.js @@ -145,6 +145,7 @@ var PermissionsHelper = { * @return A permission value defined in nsIPermissionManager. */ getPermission: function getPermission(aPrincipal, aType) { + let aURI = BrowserApp.selectedBrowser.lastURI; // Password saving isn't a nsIPermissionManager permission type, so handle // it seperately. if (aType == "password") { @@ -164,10 +165,10 @@ var PermissionsHelper = {

// Geolocation consumers use testExactPermissionForPrincipal if (aType == "geolocation") { - return Services.perms.testExactPermissionForPrincipal(aPrincipal, aType); + return Services.perms.testExactPermissionFromPrincipal(aPrincipal, aType); }

- return Services.perms.testPermissionForPrincipal(aPrincipal, aType); + return Services.perms.testPermissionFromPrincipal(aPrincipal, aType); },

/**