Remove nsTimeout::mPrincipal

RESOLVED FIXED in Firefox 51

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Ehsan, Unassigned)

Tracking

unspecified
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
This seems to not be used for anything any more.
(Reporter)

Comment 1

2 years ago
Created attachment 8788549 [details] [diff] [review]
Remove nsTimeout::mPrincipal

This seems to not be used for anything any more.
Attachment #8788549 - Flags: review?(bzbarsky)
Comment on attachment 8788549 [details] [diff] [review]
Remove nsTimeout::mPrincipal

r=me.

This used to be used for string setTimeout stuff.  But we started just using the principal of the window itself in bug 824864, which makes sense.  Given that we don't allow cross-origin setTimeout calls, this seems fine.  The interaction with document.domain is subtly different, but I don't think anyone cares.
Attachment #8788549 - Flags: review?(bzbarsky) → review+

Comment 3

2 years ago
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9f7f3b88d969
Remove nsTimeout::mPrincipal; r=bzbarsky

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9f7f3b88d969
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.