Page MenuHomePhabricator

Collect Largest Contentful Paint
Closed, ResolvedPublic

Description

It can go to the existing PaintTiming schema (although technically LargestContentfulPaint is its own API): https://wicg.github.io/largest-contentful-paint/

While it's likely to be the same as First Paint and First ContentfulPaint > 95% of the time based on our previous observations, it's important for us to collect the same metrics as Google, to let us investigate metric changes observed in the Search Console.

Event Timeline

Peter subscribed.

@cmassaro removing you as assigned to this, me or someone else can take this up soon.

I think what we should collect is the actual timing metric + element type. That way we can keep a track of the metric and have a counter in prometheus for the element type, so we can see if the element is a img/p or whatever it is.

Change 859534 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[schemas/event/secondary@master] Add largest contentful paint schema.

https://gerrit.wikimedia.org/r/859534

Change 859534 abandoned by Phedenskog:

[schemas/event/secondary@master] Add largest contentful paint schema.

Reason:

Lets add this to navtiming.

https://gerrit.wikimedia.org/r/859534

Change 859968 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[schemas/event/secondary@master] navtiming: Add cumulative layout shift and largest contentful paint.

https://gerrit.wikimedia.org/r/859968

Change 860822 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[mediawiki/extensions/NavigationTiming@master] Add largest contentful paint.

https://gerrit.wikimedia.org/r/860822

Change 860823 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[mediawiki/extensions/NavigationTiming@master] Add largest contentful paint.

https://gerrit.wikimedia.org/r/860823

Change 860822 abandoned by Phedenskog:

[mediawiki/extensions/NavigationTiming@master] Add largest contentful paint.

Reason:

Ooop, I updated the change id

https://gerrit.wikimedia.org/r/860822

Change 860823 abandoned by Phedenskog:

[mediawiki/extensions/NavigationTiming@master] Add largest contentful paint.

Reason:

Close and get correct dependencies

https://gerrit.wikimedia.org/r/860823

Change 860822 restored by Phedenskog:

[mediawiki/extensions/NavigationTiming@master] Add largest contentful paint.

https://gerrit.wikimedia.org/r/860822

Change 860826 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[mediawiki/extensions/NavigationTiming@master] Add largest contentful paint.

https://gerrit.wikimedia.org/r/860826

Change 860822 abandoned by Phedenskog:

[mediawiki/extensions/NavigationTiming@master] Add largest contentful paint.

Reason:

https://gerrit.wikimedia.org/r/860822

Change 860826 abandoned by Phedenskog:

[mediawiki/extensions/NavigationTiming@master] Add largest contentful paint.

Reason:

https://gerrit.wikimedia.org/r/860826

Change 859968 merged by jenkins-bot:

[schemas/event/secondary@master] navtiming: Add cumulative layout shift and largest contentful paint.

https://gerrit.wikimedia.org/r/859968

Change 861343 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[performance/navtiming@master] Add cumulative layout shift and largest contenful paint.

https://gerrit.wikimedia.org/r/861343

Change 861343 merged by jenkins-bot:

[performance/navtiming@master] Add cumulative layout shift and largest contenful paint.

https://gerrit.wikimedia.org/r/861343