Closed Bug 1235061 Opened 4 years ago Closed 4 years ago

Create histogram probe to measure % of users who have disabled zoomed view

Categories

(Firefox for Android :: Settings and Preferences, defect)

35 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 46
Tracking Status
firefox45 --- fixed
firefox46 --- fixed
fennec 45+ ---

People

(Reporter: Margaret, Assigned: Margaret)

References

Details

Attachments

(1 file)

We have UI telemetry to tell us if people touch this pref in settings, but it would be useful to get a picture of what % of users actually have this enabled/disabled.
Comment on attachment 8701866 [details]
MozReview Request: Bug 1235061 - Create histogram probe to measure how often users have zoomed view enabled/disabled. r=mfinkle

https://reviewboard.mozilla.org/r/29049/#review25861

Needs a data collection peer r+ too, right?

::: mobile/android/chrome/content/browser.js:609
(Diff revision 1)
> +        Telemetry.addData("FENNEC_TRACKING_PROTECTION_STATE", parseInt(BrowserApp.getTrackingProtectionState()));

Do we care that this was inside a NIGHTLY_BUILD block?
Attachment #8701866 - Flags: review?(mark.finkle) → review+
(In reply to Mark Finkle (:mfinkle) from comment #2)
> Comment on attachment 8701866 [details]
> MozReview Request: Bug 1235061 - Create histogram probe to measure how often
> users have zoomed view enabled/disabled. r=mfinkle
> 
> https://reviewboard.mozilla.org/r/29049/#review25861
> 
> Needs a data collection peer r+ too, right?

I was assuming you could play both roles here, but I'll ask vladan to take a look as well.
 
> ::: mobile/android/chrome/content/browser.js:609
> (Diff revision 1)
> > +        Telemetry.addData("FENNEC_TRACKING_PROTECTION_STATE", parseInt(BrowserApp.getTrackingProtectionState()));
> 
> Do we care that this was inside a NIGHTLY_BUILD block?

I was being lazy by not filing a separate bug - this used to be behind a Nightly flag because it was only tracking a pref that had UI on Nightly, but that changed with bug 1228090. With that change, this probe is useful for users up to release as well, since it tells us if they disabled tp for pb as well.
Comment on attachment 8701866 [details]
MozReview Request: Bug 1235061 - Create histogram probe to measure how often users have zoomed view enabled/disabled. r=mfinkle

Flagging vladan for a data collection review.
Attachment #8701866 - Flags: feedback?(vladan.bugzilla)
Comment on attachment 8701866 [details]
MozReview Request: Bug 1235061 - Create histogram probe to measure how often users have zoomed view enabled/disabled. r=mfinkle

https://reviewboard.mozilla.org/r/29049/#review25981

Note that this will be an opt-in histogram on Release
Attachment #8701866 - Flags: review+
Attachment #8701866 - Flags: review+
Attachment #8701866 - Flags: feedback?(vladan.bugzilla)
Attachment #8701866 - Flags: feedback+
https://hg.mozilla.org/integration/fx-team/rev/9e210cee056b1a3d213198b4e38679612b0bb0bc
Bug 1235061 - Create histogram probe to measure how often users have zoomed view enabled/disabled. r=mfinkle p=vladan
Should we track 45 where the zoomed view first appears (bug 1196146)?
tracking-fennec: --- → ?
https://hg.mozilla.org/mozilla-central/rev/9e210cee056b
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Comment on attachment 8701866 [details]
MozReview Request: Bug 1235061 - Create histogram probe to measure how often users have zoomed view enabled/disabled. r=mfinkle

Approval Request Comment
[Feature/regressing bug #]: Zoomed view (shipping in Fx45).

[User impact if declined]: We don't know how many people are disabling this feature.

[Describe test coverage new/current, TreeHerder]: No automated test for probe.

[Risks and why]: Low-risk, just adds telemetry probe.

[String/UUID change made/needed]: None.
Attachment #8701866 - Flags: approval-mozilla-aurora?
tracking-fennec: ? → 45+
Attachment #8701866 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
had to back this out for causing failures like https://treeherder.mozilla.org/logviewer.html#?job_id=1707444&repo=mozilla-aurora 

6:52 <@mfinkle> Tomcat|sheriffduty: https://bugzilla.mozilla.org/show_bug.cgi?id=1235061#c2
06:52 < firebot> Bug 1235061 — FIXED, margaret.leibovic@gmail.com — Create histogram probe to measure % of users who have disabled zoomed view
06:52 <@mfinkle> margaret ^
06:52 <@mfinkle> turns out, moving that code out of the NIGHTLY_BUILD block is what is breaking aurora
Flags: needinfo?(margaret.leibovic)
Landed an updated patch without the TP probe:
https://hg.mozilla.org/releases/mozilla-aurora/rev/2db713b1c2fb
Flags: needinfo?(margaret.leibovic)
You need to log in before you can comment on or make changes to this bug.