1890634 - Add metrics to understand how often Windows marks taskbar pinning as "Not Available"

Status: NEW

(Firefox :: Shell Integration, task, P3)

Comment 1

[Michael Hughes]

Attachment: data_review.txt

Comment 2

[Michael Hughes]

Attachment: Bug 1890634 - add telemetry for tab pinning r=nalexander,nrishel,chutten

Comment 3

[Roger Yang [:royang]]

Comment on attachment 9397239 [details]
data_review.txt

DATA COLLECTION REVIEW RESPONSE:

1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?

Yes, through the Glean Dictionary.

2. Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. These collections are Glean. The opt-out can be found in the product's preferences.

3. If the request is for permanent data collection, is there someone who will monitor the data over time?

[e-mail] will be responsible for the permanent collections.

4. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 2, Interaction data

5. Is the data collection request for default-on or default-off?

Default on for all channels.

6. Does the instrumentation include the addition of any new identifiers?

No

7. Is the data collection covered by the existing Firefox privacy notice?

Yes

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fb9d21d9a3f9
add telemetry for tab pinning r=nrishel

Comment 5

[Cristian Tuns]

Backed out for causing build bustages in Windows11TaskbarPinning.cpp

• Backout link
• Push with failures
• Failure Log
• Failure line: /builds/worker/checkouts/gecko/browser/components/shell/Windows11TaskbarPinning.cpp(141,19): error: lambda capture 'count' is not used [-Werror,-Wunused-lambda-capture]

Comment 6

[Michael Hughes]

I've kicked off another landing. Setting to needs more information to Nicholas to try to reland it or assign to someone else if it gets rolled back again.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/29ad5906bfb0
add telemetry for tab pinning r=nrishel

Comment 8

[Atila Butkovits]

Backed out for causing bustages at TaskbarPinningMetricsTests.cpp.

Backout link: https://hg.mozilla.org/integration/autoland/rev/8b7820907458bb85eac02869790ef45fa1e2fcf9

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=LMmyTvJdShOV2jVgfleW8g.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=29ad5906bfb007f51ff55b82ccba789e27de0695

Failure log: https://treeherder.mozilla.org/logviewer?job_id=456579360&repo=autoland&lineNumber=57020

Comment 9

[BugBot (nomail) [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.