Closed Bug 1596843 Opened 3 months ago Closed 3 months ago

Remove some unused nsIPermissionManager APIs

Categories

(Core :: Permission Manager, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: ehsan, Assigned: ehsan)

Details

Attachments

(4 files)

No description provided.
Pushed by rgurzau@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b10aec041377
Part 1: Remove the unused nsIPermissionManager.testPermissionFromWindow() method; r=baku
https://hg.mozilla.org/integration/autoland/rev/8b2220bc47af
Part 2: Stop exposing nsIPermissionManager.removePermissionsWithAttributes(); r=baku
https://hg.mozilla.org/integration/autoland/rev/a9827fae8655
Part 3: Remove the unused nsIPermissionManager.updateExpireTime() method; r=baku
https://hg.mozilla.org/integration/autoland/rev/69ee727d5b45
Part 4: Remove some native only methods from nsIPermissionManager; r=baku
Backout by rgurzau@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/de185bb7bdd5
Backed out 4 changesets for XPCShellTestsTests build bustage ona CLOSED TREE.

: Ehsan Hi, tried to land these changes to test them but got some failures and a backout was needed.

These are the failures:

  • XPCShellTestsTests::testChild TEST-UNEXPECTED-FAIL: TH link | Log link

  • Assertion failure at nsPermissionManager.cpp: Th link | Log link

  • application crashed [@ nsPermissionManager::SetPermissionsWithKey(nsTSubstring<char> const&, nsTArray<IPC::Permission>&)] TH link | Log link

Flags: needinfo?(ehsan)
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/88e8383dc3e4
Part 1: Remove the unused nsIPermissionManager.testPermissionFromWindow() method; r=baku
https://hg.mozilla.org/integration/autoland/rev/691397c2fde6
Part 2: Stop exposing nsIPermissionManager.removePermissionsWithAttributes(); r=baku
https://hg.mozilla.org/integration/autoland/rev/b9061f93d045
Part 3: Remove the unused nsIPermissionManager.updateExpireTime() method; r=baku
https://hg.mozilla.org/integration/autoland/rev/48577632f4b3
Part 4: Remove some native only methods from nsIPermissionManager; r=baku
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/83096c9e152d
Part 1: Remove the unused nsIPermissionManager.testPermissionFromWindow() method; r=baku
https://hg.mozilla.org/integration/autoland/rev/70e1692d29f5
Part 2: Stop exposing nsIPermissionManager.removePermissionsWithAttributes(); r=baku
https://hg.mozilla.org/integration/autoland/rev/21bfefb23045
Part 3: Remove the unused nsIPermissionManager.updateExpireTime() method; r=baku
https://hg.mozilla.org/integration/autoland/rev/03b7360a15dd
Part 4: Remove some native only methods from nsIPermissionManager; r=baku
You need to log in before you can comment on or make changes to this bug.