1898649 - Disable cert_verifier.cert_trust_evaluation_time metric on Android

Status: RESOLVED FIXED

(Core :: Security: PSM, task)

Description

[Jeff Muizelaar [:jrmuizel]]

This metric is recorded 600 times during a browser restore of cnn.com and looks to be the most frequently recorded metric in the parent after performance.clone.deserialize from bug 1898515.

Comment 1

[Jeff Muizelaar [:jrmuizel]]

Attachment: Bug 1898649. Disable cert_verifier.cert_trust_evaluation_time metric on Android.

This metric is recorded 600 times during a browser restore of cnn.com
and looks to be the most frequently recorded metric in the parent after
performance.clone.deserialize from bug 1898515.

Comment 2

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 9403748 [details]
Bug 1898649. Disable cert_verifier.cert_trust_evaluation_time metric on Android.

Beta/Release Uplift Approval Request

• User impact if declined: We spend a lot of extra cpu time and do a lot of extra disk writes during page load.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This just disables recording the cert_verifier.cert_trust_evaluation_time metric on Android it should otherwise have no functional change.
• String changes made/needed:
• Is Android affected?: Yes

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d4a540b2d531
Disable cert_verifier.cert_trust_evaluation_time metric on Android.

Comment 4

[Jeff Muizelaar [:jrmuizel]]

I got a verbal go-ahead from jschanck over slack before landing this.

Comment 5

[tszentpeteri]

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

Comment 6

[Donal Meehan [:dmeehan]]

Comment on attachment 9403748 [details]
Bug 1898649. Disable cert_verifier.cert_trust_evaluation_time metric on Android.

Approved for 127.0b7

Comment 7

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/542b49746d07

Comment 8

[Donal Meehan [:dmeehan]]

Comment on attachment 9403748 [details]
Bug 1898649. Disable cert_verifier.cert_trust_evaluation_time metric on Android.

Approved for 126.0.1

Comment 9

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-release/rev/6c033deedc28