1623417 - Remove usage of GetURI in nsPermission.cpp

Status: RESOLVED FIXED

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

Description

[Sebastian Streich [:sstreich]]

See Metabug

Comment 1

[Sebastian Streich [:sstreich]]

Attachment: Bug 1623417 - Refactor nsPermission.cpp r=ckerschb

---

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

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.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a2bff0eef797
Refactor nsPermission.cpp r=ckerschb

Comment 4

[Noemi Erli[:noemi_erli]]

Backed out changeset a2bff0eef797 (Bug 1623417) for causing xpcshell failures in test_permmanager_matchesuri.js CLOSED TREE

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=298441911&resultStatus=pending%2Crunning%2Ctestfailed%2Cbusted%2Cexception&revision=a2bff0eef797dc2f2f40779eb08ccfae0ceeb27f&searchStr=linux%2C18.04%2Cx64%2Cquantumrender%2Cdebug%2Cxpcshell%2Ctests%2Ctest-linux1804-64-qr%2Fdebug-xpcshell-e10s-6%2Cx%28x6%29

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=298435427&repo=autoland&lineNumber=3116

Backout: https://hg.mozilla.org/integration/autoland/rev/475d45a0aea17bf1be100ca30b41db74ef7cbe1b

Comment 5

[Sebastian Streich [:sstreich]]

Fixed the failure, should work now :)

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/70da39008835
Refactor nsPermission.cpp r=ckerschb

Comment 7

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/70da39008835