Commit 5c8c045e authored by Andrei Lazar's avatar Andrei Lazar Committed by Matthew Finkel
Browse files

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
parent cf72c982

mobile/android/chrome/content/PermissionsHelper.js

+3 −2
Original line number Original line Diff line number Diff line
@@ -145,6 +145,7 @@ var PermissionsHelper = { 
   * @return A permission value defined in nsIPermissionManager.

   * @return A permission value defined in nsIPermissionManager.

   */

   */

  getPermission: function getPermission(aPrincipal, aType) {

  getPermission: function getPermission(aPrincipal, aType) {

    let aURI = BrowserApp.selectedBrowser.lastURI;

    // Password saving isn't a nsIPermissionManager permission type, so handle

    // Password saving isn't a nsIPermissionManager permission type, so handle

    // it seperately.

    // it seperately.

    if (aType == "password") {

    if (aType == "password") {
@@ -164,10 +165,10 @@ var PermissionsHelper = { 




    // Geolocation consumers use testExactPermissionForPrincipal

    // Geolocation consumers use testExactPermissionForPrincipal

    if (aType == "geolocation") {

    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);

  },

  },





  /**

  /**