Remove nsTimeout::mPrincipal

RESOLVED FIXED in Firefox 51

Status

()

Core
DOM
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: Away for a while, Unassigned)

Tracking

unspecified
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

a year ago
This seems to not be used for anything any more.
(Reporter)

Comment 1

a year 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

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

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9f7f3b88d969
Status: NEW → RESOLVED
Last Resolved: a year 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.