[prev in list] [next in list] [prev in thread] [next in thread]
List: webkit-dev
Subject: Re: [webkit-dev] Suggesting to enable paint timing by default
From: Noam Rosenthal <noam () webkit ! org>
Date: 2020-05-12 4:53:16
Message-ID: CAGttnEXdRG0gk9E-sXMzn24R4Y65hs0+sd2keP9knF_oiF9ZnQ () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
On Tue, May 12, 2020 at 1:36 AM Maciej Stachowiak <mjs@apple.com> wrote:
>
> I noticed from comments in one of the Radars that the patch may result in
> an additional "fake paint", so it should probably be performance tested.
> Have you done any testing?
>
I've tested it locally, I haven't noticed any significant side effect,
because in complex situations the fake paint only happens once per page and
bails early once contentfulness is detected. but I can run any additional
test needed.
> We'll likely want to A/B some of Apple's page load speed benchmarks.
>
A/B testing load speed sounds sensible. How do we go about doing that?
>
> I'd like to hear from others on maturity of the spec and readiness of the
> code.
>
> - Maciej
>
> On May 11, 2020, at 11:44 AM, Noam Rosenthal <noam@webkit.org> wrote:
>
> Following the discussion with Simon in
> https://bugs.webkit.org/show_bug.cgi?id=78011
>
> Since we have a pretty stable spec (https://w3c.github.io/paint-timing/),
> lots of passing web platform tests, other browser vendor support and a
> working implementation of first contentful paint, I am planning to submit a
> patch to enable paint timing by default.
>
> https://bugs.webkit.org/show_bug.cgi?id=211736
>
> Any objections? Other thoughts?
> _______________________________________________
> webkit-dev mailing list
> webkit-dev@lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev
>
>
>
[Attachment #5 (text/html)]
<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" \
class="gmail_attr">On Tue, May 12, 2020 at 1:36 AM Maciej Stachowiak <<a \
href="mailto:mjs@apple.com">mjs@apple.com</a>> wrote:<br></div><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: \
break-word;"><div><br></div>I noticed from comments in one of the Radars that the \
patch may result in an additional "fake paint", so it should probably be performance \
tested. Have you done any testing? </div></blockquote><div>I've tested it \
locally, I haven't noticed any significant side effect, because in complex \
situations the fake paint only happens once per page and bails early once \
contentfulness is detected. but I can run any additional test needed.<br> \
</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px \
solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: \
break-word;">We'll likely want to A/B some of Apple's page load speed \
benchmarks.</div></blockquote><div>A/B testing load speed sounds sensible. How do we \
go about doing that?<br> </div><blockquote class="gmail_quote" style="margin:0px 0px \
0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div \
style="overflow-wrap: break-word;"><div><br></div><div>I'd like to hear from others \
on maturity of the spec and readiness of the \
code.</div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px \
0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div \
style="overflow-wrap: break-word;"><div><br></div><div> - \
Maciej<br><div><br><blockquote type="cite"><div>On May 11, 2020, at 11:44 AM, Noam \
Rosenthal <<a href="mailto:noam@webkit.org" \
target="_blank">noam@webkit.org</a>> wrote:</div><br><div><div dir="ltr">Following \
the discussion with Simon in <a href="https://bugs.webkit.org/show_bug.cgi?id=78011" \
target="_blank">https://bugs.webkit.org/show_bug.cgi?id=78011</a><br><br>Since we \
have a pretty stable spec (<a href="https://w3c.github.io/paint-timing/" \
target="_blank">https://w3c.github.io/paint-timing/</a>), lots of passing web \
platform tests, other browser vendor support and a working implementation of first \
contentful paint, I am planning to submit a patch to enable paint timing by default. \
<div><br><a href="https://bugs.webkit.org/show_bug.cgi?id=211736" \
target="_blank">https://bugs.webkit.org/show_bug.cgi?id=211736</a><br></div><div><br></div><div>Any \
objections? Other thoughts?<br></div></div> \
_______________________________________________<br>webkit-dev mailing list<br><a \
href="mailto:webkit-dev@lists.webkit.org" \
target="_blank">webkit-dev@lists.webkit.org</a><br><a \
href="https://lists.webkit.org/mailman/listinfo/webkit-dev" \
target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br></div></blockquote></div><br></div></div></blockquote></div></div>
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic