
<html><body><div style="font-family: arial,helvetica,sans-serif; font-size: 12pt; color: #000000"><div><br></div><hr id="zwchr"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><b>From: </b>"Mark Finkle" <mfinkle@mozilla.com><br><b>To: </b>"Gavin Sharp" <gavin@gavinsharp.com><br><b>Cc: </b>"Mark Hammond" <mhammond@mozilla.com>, "Firefox Dev" <firefox-dev@mozilla.org><br><b>Sent: </b>Wednesday, September 24, 2014 9:43:43 PM<br><b>Subject: </b>Re: Changes to how default permissions are setup in Firefox 35<br><div><br></div><div style="font-family: Times New Roman; font-size: 12pt; color: #000000"><div><br></div><div><br></div><hr id="zwchr"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;">This change has no impact on what Firefox users can do - the<br>permissions UI and behavior is not changing. The only visible behavior<br>change for users is that you won't lose the default permissions when<br>you clear "Site Permissions" in "Clear Recent History".</blockquote><div>Do these changes affect Firefox on Desktop only? If not, what bugs are filed for other apps?<br></div><br></div></blockquote><div>Sorry I didn't catch in my review that there was a use of PermissionsUtils in mobile so add-on/app/LWT installation (perhaps only in new profiles depending on how the exception propagates) is probably broken. I filed bug 1072744[1] for mobile but perhaps we should just backout bug 1050080 for now until that's resolved. I will file a bug for thunderbird and seamonkey too.<br></div><div><br></div><div>I also noticed that the default location for the the default permission file is hard-coded to include "/browser/" in the path which seems unusual for code living in <span class="file-label-name">extensions/cookie/nsPermissionManager.cpp</span>.<br></div><div><br></div><div>[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1072744<br></div></div></body></html>