Closed Bug 1636148 Opened 5 months ago Closed 4 months ago

Make calling nsIPrincipal->GetURI a clang-error

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla78
Tracking Status
firefox78 --- fixed

People

(Reporter: sstreich, Assigned: sstreich)

References

(Blocks 1 open bug)

Details

(Whiteboard: [domsecurity-active])

Attachments

(2 files)

We've come far to remove all Callsites of Geturi in "c-land". To make shure no new callers can be added until we we've cleaned up js land (and thus can just remove the interface) - I want to change the clang plugin to produce a build error instead of a warning.

Assignee: nobody → sstreich
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: [domsecurity-active]
Pushed by dluca@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6fda7ca2484d
"BandAid" - Cast Remaining Callers To BasePrincipal r=ckerschb
https://hg.mozilla.org/integration/autoland/rev/fca4a9808bd9
Disable Building Principal->GetURI r=ckerschb

Sorry! should work now :)

Flags: needinfo?(sstreich)
Pushed by dluca@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b328a689b674
"BandAid" - Cast Remaining Callers To BasePrincipal r=ckerschb,geckoview-reviewers,agi
https://hg.mozilla.org/integration/autoland/rev/1a7e54a71d1d
Disable Building Principal->GetURI r=ckerschb
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
See Also: → 1638288
You need to log in before you can comment on or make changes to this bug.