-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FF65 Reporting API updates #24112
FF65 Reporting API updates #24112
Conversation
"name": "dom.reporting.enabled", | ||
"value_to_set": "true" | ||
} | ||
] | ||
}, | ||
"firefox_android": "mirror", | ||
"ie": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Further down is a toJSON
method. The spec IDL includes this in DeprecationReportBody
and the parent class ReportBody base class too. But this method is NOT declared in FF in DeprecationReportBody
- would it still be updated here because it is inherited?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If Firefox doesn't support it, then keeping it set to false
seems correct to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Hamish! 👍
This updates FF features for the Reporting API
Most of this was implemented in FF65 in the "legacy v0 API" in https://bugzilla.mozilla.org/show_bug.cgi?id=1492036 behind pref. Where supported,
to_JSON
and worker support added in https://bugzilla.mozilla.org/show_bug.cgi?id=1630947Note, FF does not support:
There's a new header being added in #24047, so this is "backfilling"
Related docs work tracked in mdn/content#35279