add telemetry comparing system and geoIP based location for Windows

RESOLVED FIXED in Firefox 37

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Gavin, Assigned: markh)

Tracking

(Blocks 1 bug)

Trunk
Firefox 38
All
Windows 8
Points:
1
Dependency tree / graph
Bug Flags:
firefox-backlog +
in-testsuite +
qe-verify -

Firefox Tracking Flags

(firefox37+ fixed, firefox38+ fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

The Windows version of bug 1121340.
We should get this in 37 to get an earlier decision on whether to pursue bug 1115106.
Flags: needinfo?(mhammond)
Mark can you take this?
Points: --- → 1
Flags: qe-verify-
Flags: firefox-backlog+
A fairly simple patch given the foundation was already set by the OSX patch.

Note however that as Windows also uses nsIGfxInfo2, Windows builds are going to start reporting the _OSX probes as there was no specific check for OSX in nsSearchService (sadly, I expected the use of nsIGfxInfo2 was a hack for Mac only and Windows would do something different - but I was wrong!)  Thus we should get this landed and uplifted ASAP.
Assignee: nobody → mhammond
Status: NEW → ASSIGNED
Flags: needinfo?(mhammond)
Attachment #8563146 - Flags: review?(gavin.sharp)
Iteration: --- → 38.3 - 23 Feb
ack - I neglected to change some comments in the test which are fixed here.
Attachment #8563146 - Attachment is obsolete: true
Attachment #8563146 - Flags: review?(gavin.sharp)
Attachment #8563149 - Flags: review?(gavin.sharp)
Comment on attachment 8563149 [details] [diff] [review]
0001-Bug-1130132-add-telemetry-to-determine-how-consisten.patch

Oops, yes.
Attachment #8563149 - Flags: review?(gavin.sharp) → review+
Attachment #8563149 - Flags: approval-mozilla-aurora+
Note to sheriffs - to avoid Aurora recording bad telemetry data, please uplift this patch at the same time as bug 1121340 (but note that this bug depends on that)
Whiteboard: [sheriff note in comment 7]
https://hg.mozilla.org/mozilla-central/rev/a4f8a5a7e3eb
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 38
You need to log in before you can comment on or make changes to this bug.