Extend telemetry period for embedded YouTube Flash videos from FF 48 to 53

RESOLVED FIXED in Firefox 48

Status

()

Core
Plug-ins
RESOLVED FIXED
2 years ago
8 months ago

People

(Reporter: cpeterson, Assigned: cpeterson)

Tracking

Trunk
mozilla48
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox45 unaffected, firefox46 unaffected, firefox47 unaffected, firefox48 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Created attachment 8731366 [details] [diff] [review]
extend-youtube-telemetry-period.patch

The YOUTUBE_REWRITABLE_EMBED_SEEN and YOUTUBE_NONREWRITABLE_EMBED_SEEN telemetry probes expire in FF 48. FF 53 is the last Nightly version of 2016 and will hit the GA channel around April 2017. The choice of version 53 is somewhat arbitrary, but long enough to get us data through the beginning of next year.
Attachment #8731366 - Flags: review?(kyle)
Comment on attachment 8731366 [details] [diff] [review]
extend-youtube-telemetry-period.patch

Review of attachment 8731366 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, assuming I'm ok to review this while not being a metrics peer (which I think is ok since we're just changing, not adding?)
Attachment #8731366 - Flags: review?(kyle) → review+
(Assignee)

Comment 2

2 years ago
Comment on attachment 8731366 [details] [diff] [review]
extend-youtube-telemetry-period.patch

Review of attachment 8731366 [details] [diff] [review]:
-----------------------------------------------------------------

Benjamin, we're collecting telemetry about our YouTube Flash embed rewriter. Our probes, YOUTUBE_REWRITABLE_EMBED_SEEN and YOUTUBE_NONREWRITABLE_EMBED_SEEN, expire in FF 48, but the feature rollout was delayed so we'd like to extend the telemetry period.

FF 53 is the last Nightly version of 2016 and will hit the GA channel around April 2017. The choice of version 53 is somewhat arbitrary, but long enough to get us data through the beginning of next year. That will give us current data when we're making other product decisions about NPAPI plugins.
Attachment #8731366 - Flags: review?(benjamin)

Comment 3

2 years ago
Comment on attachment 8731366 [details] [diff] [review]
extend-youtube-telemetry-period.patch

Chris, you are still actively monitoring this data? data-review=me if that's the case. If you're actively monitoring and believe this is valuable, why don't we make it opt-out?
Attachment #8731366 - Flags: review?(benjamin) → feedback+
(Assignee)

Comment 4

2 years ago
(In reply to Benjamin Smedberg  [:bsmedberg] from comment #3)
> Chris, you are still actively monitoring this data? data-review=me if that's
> the case. If you're actively monitoring and believe this is valuable, why
> don't we make it opt-out?

Yes because the Flash embed rewriter has not hit the Release channel yet. We want to see if this probe changes after it does.

I will make this opt-out, starting in 48. Changing from opt-in to opt-out users may affect the data, so we will just need to remember that we may see a discontinuity in 48's results.

Comment 5

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/ea27fb411f21

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ea27fb411f21
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox48: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
(Assignee)

Updated

8 months ago
Depends on: 1354778
You need to log in before you can comment on or make changes to this bug.