Closed Bug 1528916 Opened 3 years ago Closed 3 years ago

Explicitly set LoadInfo for "about:crashcontent" and "about:crashparent"

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: ckerschb, Assigned: ckerschb)

References

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

Within Bug 1507702 we introduced a shortcut for "about:crashcontent" and "about:crashparent" within nsAboutProtocolHandler.cpp, but we didn't explicitly set the loadinfo for that channel.

Within Bug 1528677 I want to remove all nullchecks for the loadinfo, since we should have a loadinfo on all channels by now.

Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: [domsecurity-active]

(In reply to Christoph Kerschbaumer [:ckerschb] from comment #0)

for "about:crashcontent" and "about:crashparent" within nsAboutProtocolHandler.cpp
I meant to say within nsAboutRedirector.cpp

Pushed by mozilla@christophkerschbaumer.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f3197be1dd89
Explicitly set LoadInfo for about:crashcontent and about:crashparent. r=gijs

Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.