'Re: [webkit-dev] Suggesting to enable paint timing by default'

[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:       Maciej Stachowiak <mjs () apple ! com>
Date:       2020-05-12 7:28:29
Message-ID: 4969DBD8-D9AF-4C89-9FF3-5EECAAA108D6 () apple ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

> On May 11, 2020, at 9:53 PM, Noam Rosenthal <noam@webkit.org> wrote:
> 
> 
> 
> On Tue, May 12, 2020 at 1:36 AM Maciej Stachowiak <mjs@apple.com \
> <mailto: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?

Unfortunately our page load speed benchmarks are not public because they incorporate \
captured page content, which we can't freely redistribute.

A helpful person from Apple may be able to set up an A/B test for this patch.

 - Maciej

[Attachment #5 (unknown)]

<html><head><meta http-equiv="Content-Type" content="text/html; \
charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; \
line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote \
type="cite" class=""><div class="">On May 11, 2020, at 9:53 PM, Noam Rosenthal &lt;<a \
href="mailto:noam@webkit.org" class="">noam@webkit.org</a>&gt; wrote:</div><br \
class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div \
dir="ltr" class=""><br class=""></div><br class=""><div class="gmail_quote"><div \
dir="ltr" class="gmail_attr">On Tue, May 12, 2020 at 1:36 AM Maciej Stachowiak &lt;<a \
href="mailto:mjs@apple.com" class="">mjs@apple.com</a>&gt; wrote:<br \
class=""></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;" class=""><div class=""><br class=""></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 class="">I've tested it locally, I haven't noticed any \
significant side effect, because in complex situations&nbsp;the fake paint only \
happens once per page and bails early once contentfulness is detected. but I can run \
any additional test needed.<br class="">&nbsp;</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;" \
class="">We'll likely want to A/B some of Apple's page load speed \
benchmarks.</div></blockquote><div class="">A/B testing load speed sounds sensible. \
How do we go about doing that?<br class=""></div></div></div></div></blockquote><br \
class=""></div><div>Unfortunately our page load speed benchmarks are not public \
because they incorporate captured page content, which we can't freely \
redistribute.</div><div><br class=""></div><div>A helpful person from Apple may be \
able to set up an A/B test for this patch.</div><div><br class=""></div><div>&nbsp;- \
Maciej</div><br class=""></body></html>

_______________________________________________
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] 