Closed Bug 1620402 Opened 5 months ago Closed 4 months ago

Remove usage of GetURI in nsPermissionManager.cpp

Categories

(Core :: DOM: Security, task, P2)

task

Tracking

()

RESOLVED FIXED
mozilla77
Tracking Status
firefox77 --- fixed

People

(Reporter: sstreich, Assigned: sstreich)

References

(Blocks 1 open bug)

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

See Metabug

Assignee: nobody → sstreich
No longer depends on: 1620292
Blocks: 1620441
No longer blocks: 1620441
Status: NEW → ASSIGNED
Whiteboard: [dom-security-active] → [domsecurity-active]
Blocks: 1621299
No longer blocks: 1621299
Pushed by rgurzau@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/848221b2e156
Refactor nsPermissionManager.cpp r=ckerschb
Backout by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/059be1f4b4ed
Backed out changeset 848221b2e156 for causing several test failures.

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:sstreich, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(sstreich)

Updated the Patch, got a green try and flagged for checkin

Flags: needinfo?(sstreich)
Pushed by apavel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f5c2549f8886
Refactor nsPermissionManager.cpp r=ckerschb
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.