Disable devicelight, deviceproximity and userproximity events from stable

RESOLVED FIXED in Firefox 62

Status

()

enhancement
P2
normal
RESOLVED FIXED
a year ago
10 months ago

People

(Reporter: jkt, Assigned: jkt)

Tracking

({dev-doc-complete, site-compat})

62 Branch
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

a year ago
In Bug 1359076 we disabled devicelight, deviceproximity and userproximity for Nightly and Early Beta.

Firefox started firing telemetry in late beta for these events along with the other related events Motion and Orientation.

The former that we plan to disable don't even seem to feature in this telemetry, I suspect due to the insignificant number of calls to this telemetry.

There haven't been any other reports calling for this feature so as planned in 62 we should disable these non standard events from stable.
Priority: -- → P2

Comment 2

a year ago
mozreview-review
Comment on attachment 8976492 [details]
Bug 1462308 - Disable all Device Sensor events except orientation by default for stable.

https://reviewboard.mozilla.org/r/244612/#review250816
Attachment #8976492 - Flags: review?(bkelly) → review+

Comment 3

a year ago
Pushed by jkingston@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/98a2883d9640
Disable all Device Sensor events except orientation by default for stable. r=bkelly

Comment 5

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/98a2883d9640
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Assignee

Updated

11 months ago
Depends on: 1359076
Ha, I didn't have much to do here because I did most of it in the previous related bug.

For this work, I've made sure the compat data is up-to-date:
https://github.com/mdn/browser-compat-data/pull/2505

And moved the release notes info to apply to Firefox 62:
https://developer.mozilla.org/en-US/Firefox/Releases/62#APIs_2

Does this look OK?
Flags: needinfo?(jkt)
Assignee

Comment 7

10 months ago
This looks great thank you!
Flags: needinfo?(jkt)
You need to log in before you can comment on or make changes to this bug.