Closed Bug 1674779 Opened 4 years ago Closed 4 years ago

Improve documentation of GV_PAGE_LOAD_MS (describe back navigation and bimodality)

Categories

(Core :: Performance, enhancement)

ARM
Android
enhancement

Tracking

()

RESOLVED FIXED
84 Branch
Tracking Status
firefox84 --- fixed

People

(Reporter: acreskey, Assigned: acreskey)

References

Details

Attachments

(1 file)

In Bug 1624360 we investigated the bimodality and suspiciously fast results of GV_PAGE_LOAD_MS probe in Fenix.

A primary cause was found to be the inclusion of back navigation. In particular, when this is performed via the BFCache the results are generally <50ms even on slower Android devices.

This bug is to improve the metrics.yaml and histogram.json documentation that teams making use of this probe can better understand the results.

Blocks: 1624360

Add notes to metrics.yaml and Histograms.json to describe a source of bimodality in this probe.

Hi Dexter, I put a patch with some improvements to the probe description. Can you please review or suggest someone else?

Flags: needinfo?(alessio.placitelli)

Whoops, sorry for the delay. For some reason I did not receive the email notification for this.

Flags: needinfo?(alessio.placitelli)
Pushed by ncsoregi@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b130bd668e90
Improve documentation of GV_PAGE_LOAD_MS (describe back navigation and bimodality) r=Dexter

I am thinking replace the '<' with LT.

Sorry about that and thanks for the backout.
I left two trailing spaces in there.

Flags: needinfo?(acreskey)
Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b3846467a384
Improve documentation of GV_PAGE_LOAD_MS (describe back navigation and bimodality) r=Dexter
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: