1229486 - FX_TABLETMODE_PAGE_LOAD reports the same counts for "desktop" and "tablet" mode

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Vladan Djeric (:vladan)]

On my Win7 Lenovo W520 laptop, I see the same counts for the "desktop" and "tablet" keys of the FX_TABLETMODE_PAGE_LOAD histogram.

I never unplug this laptop from its 2 external monitors, and it does not have a touch screen, so it should never be in "tablet mode".

Reproduces on Nightly 45 and Aurora 44.

Comment 1

[:Gijs (he/him)]

Vladan and I debugged this over IRC - the TL;DR is that add(0) on a histogram does increment that histogram's "0" bucket, so it's not the same as just not calling it. To ensure we don't submit data for this histogram on non-win10, we should avoid finish() actually calling add() for the histogram.

Comment 2

[:Gijs (he/him)]

Attachment: MozReview Request: Bug 1229486 - fix submission of tablet mode histogram on non-win10, r?jaws

Bug 1229486 - fix submission of tablet mode histogram on non-win10, r?jaws

Comment 3

[:Gijs (he/him)]

Jared: Tested on OSX, would be good if you can verify this does still submit stuff on win10 (it should, but there we go).

Comment 4

[:Gijs (he/him)]

I verified that this is still working on Win10.

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8694660 [details]
MozReview Request: Bug 1229486 - fix submission of tablet mode histogram on non-win10, r?jaws

https://reviewboard.mozilla.org/r/26825/#review25243

I didn't get a chance to test this locally but the patch looks good to me.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/eb53c7d6b494

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/eb53c7d6b494

Comment 8

[:Gijs (he/him)]

Comment on attachment 8694660 [details]
MozReview Request: Bug 1229486 - fix submission of tablet mode histogram on non-win10, r?jaws

Approval Request Comment
[Feature/regressing bug #]: telemetry for tablet mode on win10
[User impact if declined]: nonsensical telemetry submitted for other versions of Windows and other OSes
[Describe test coverage new/current, TreeHerder]: no
[Risks and why]: very low, correctness fix so we don't touch the relevant histogram except on win10+
[String/UUID change made/needed]: no

Comment 9

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8694660 [details]
MozReview Request: Bug 1229486 - fix submission of tablet mode histogram on non-win10, r?jaws

Improve our data, taking it.

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/1ce9e7a3091a

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/d90ed2651b3d

Comment 12

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/d90ed2651b3d

Comment 13

[Mayur Patil]

[bugday-20160323]

Status: RESOLVED,FIXED -> UNVERIFIED

Comments:
STR: Not clear.
Developer specific testing

Component: 
Name			Firefox
Version			46.0b9
Build ID		20160322075646
Update Channel	        beta
User Agent		Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS			Windows 7 SP1 x86_64

Expected Results: 
Developer specific testing

Actual Results: 
As expected

Comment 14

[Mayur Patil]

[bugday-20160323]

Status: RESOLVED,FIXED -> UNVERIFIED

Comments:
STR: Not clear.
Developer specific testing

Component: 
Name			Firefox
Version			46.0b9
Build ID		20160322075646
Update Channel	beta
User Agent		Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS				Windows 7 SP1 x86_64

Expected Results: 
Developer specific testing

Actual Results: 
As expected