Closed
Bug 990220
Opened 11 years ago
Closed 11 years ago
Enable navigator.sendBeacon by default on desktop and Android
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla31
Tracking | Status | |
---|---|---|
relnote-firefox | --- | 31+ |
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
1.00 KB,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
(Please add more bugs to the dependency list for things that we need to fix before enabling this by default.)
I don't think there's anything preventing us from flipping this on for desktop nightlies.
Comment 3•11 years ago
|
||
Spec is stable enough? /me would really like to see this enabled in order to start warn about
use of sync XHR during unload
Assignee | ||
Comment 4•11 years ago
|
||
Even if the spec is not stable enough, I think it might be a good idea to unhide and publicize this for non-release builds to allow people to start experimenting with it.
The spec has been stable for quite a while. The intent was for bug 936340 to enable sendBeacon by default, so we should just go for it.
Assignee | ||
Comment 6•11 years ago
|
||
(In reply to comment #5)
> The spec has been stable for quite a while. The intent was for bug 936340 to
> enable sendBeacon by default, so we should just go for it.
Sounds good to me then!
Assignee | ||
Comment 7•11 years ago
|
||
I'll file a follow-up for b2g as well.
Assignee | ||
Updated•11 years ago
|
Summary: Enable navigator.sendBeacon by default → Enable navigator.sendBeacon by default on desktop and Android
Assignee | ||
Comment 8•11 years ago
|
||
(Filed bug 990270 for b2g)
Attachment #8399656 -
Flags: review?(jonas) → review+
Assignee | ||
Comment 9•11 years ago
|
||
Assignee | ||
Comment 10•11 years ago
|
||
Richard, I'd appreciate if you could send an intent to ship email to dev-platform. Thanks!
Flags: needinfo?(rlb)
Assignee | ||
Updated•11 years ago
|
Keywords: dev-doc-needed
Comment 11•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Comment 12•11 years ago
|
||
Sent intent to ship message to dev-platform.
Updated•11 years ago
|
Flags: needinfo?(rlb)
Comment 14•11 years ago
|
||
I added "navigator.sendBeacon enabled by default" to the release notes. Let me know if you want a better wording.
I will add a link to the documentation once it is available.
Comment 15•8 years ago
|
||
Updated the following pages. Sorry this one slipped through the cracks way back when!
https://developer.mozilla.org/en-US/docs/Web/API/Navigator/sendBeacon
https://developer.mozilla.org/en-US/Firefox/Releases/31
Keywords: dev-doc-needed → dev-doc-complete
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•