Closed Bug 990220 Opened 10 years ago Closed 10 years ago

Enable navigator.sendBeacon by default on desktop and Android

Categories

(Core :: DOM: Core & HTML, defect)

x86
macOS
defect
Not set
normal

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)

      No description provided.
(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.
Spec is stable enough? /me would really like to see this enabled in order to start warn about
use of sync XHR during unload
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.
(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!
Attached patch Patch (v1)Splinter Review
I'll file a follow-up for b2g as well.
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #8399656 - Flags: review?(jonas)
Summary: Enable navigator.sendBeacon by default → Enable navigator.sendBeacon by default on desktop and Android
(Filed bug 990270 for b2g)
No longer depends on: 988107
Richard, I'd appreciate if you could send an intent to ship email to dev-platform.  Thanks!
Flags: needinfo?(rlb)
Keywords: dev-doc-needed
https://hg.mozilla.org/mozilla-central/rev/0483fe9d0b74
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Sent intent to ship message to dev-platform.
Flags: needinfo?(rlb)
We will want this in the 31 release notes.
relnote-firefox: --- → ?
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.
Blocks: 980902
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: