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

2020-05-12 Thread Noam Rosenthal
On Tue, May 12, 2020 at 10:28 AM Maciej Stachowiak  wrote:

>
>
> On May 11, 2020, at 9:53 PM, Noam Rosenthal  wrote:
>
>
>
> On Tue, May 12, 2020 at 1:36 AM Maciej Stachowiak  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.
>
 Understood. I'd be happy to facilitate/support/do what's necessary to move
this forward.
Thanks!
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


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

2020-05-12 Thread Maciej Stachowiak


> On May 11, 2020, at 9:53 PM, Noam Rosenthal  wrote:
> 
> 
> 
> On Tue, May 12, 2020 at 1:36 AM Maciej Stachowiak  > 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

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev