2057731 - Add telemetry for the "global stylesheet not found" case.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement, P3)

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 2057731 - Add telemetry for the global stylesheet cache not finding the expected stylesheet. r=dholbert,Gijs

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/87f604e7f075
https://hg.mozilla.org/integration/autoland/rev/bb0029a5b903
Add telemetry for the global stylesheet cache not finding the expected stylesheet. r=dholbert,layout-reviewers

Comment 3

[Serban Stanca [:SerbanS]]

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

Comment 4

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 2057731 - Add telemetry for the global stylesheet cache not finding the expected stylesheet.

Original Revision: https://phabricator.services.mozilla.com/D314213

Comment 5

[Phabricator Automation]

firefox-beta Uplift Approval Request

• User impact if declined/Reason for urgency: Gives us some visibility on the error condition instead of a crash.
• Code covered by automated testing?: no
• Fix verified in Nightly?: no
• Needs manual QE testing?: no
• Steps to reproduce for manual QE testing:
• Risk associated with taking this patch: low
• Explanation of risk level: Adds one telemetry probe.
• String changes made/needed?: None
• Is Android affected?: yes

Comment 6

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/1d7be8374b28
https://hg.mozilla.org/releases/mozilla-beta/rev/abff8cf729e2

Comment 7

[Daniel Holbert [:dholbert]]

Did you want to request uplift to release, too?

Comment 8

[Emilio Cobos Álvarez [:emilio]]

Yeah I tried but lando failed to do it probably because the dependency wasn't there yet.

Comment 9

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 2057731 - Add telemetry for the global stylesheet cache not finding the expected stylesheet.

Original Revision: https://phabricator.services.mozilla.com/D314213

Comment 10

[Phabricator Automation]

firefox-release Uplift Approval Request

• User impact if declined/Reason for urgency: Gives us some visibility on the error condition instead of a crash.
• Code covered by automated testing?: no
• Fix verified in Nightly?: no
• Needs manual QE testing?: no
• Steps to reproduce for manual QE testing:
• Risk associated with taking this patch: low
• Explanation of risk level: Adds one telemetry probe.
• String changes made/needed?: None
• Is Android affected?: yes

Comment 11

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/b7860a1dbc4e
https://hg.mozilla.org/releases/mozilla-release/rev/d43cb0f0b8e8

Comment 12

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-esr153/rev/f07803e1aa58