Re: RFR: 8260165: CSSFilterTest.testCSSFilterRendering system test fails

2021-02-26 Thread Guru Hb
On Mon, 22 Feb 2021 11:55:00 GMT, Arun Joseph wrote: > Issue: Initial layout delay was removed and layout() is called from > layoutTimer instead of WebPage::prePaint(). > > Fix: Re-introduce the initial layout delay. Tested on Linux and Windows, Looks good to me. - Marked as

Re: RFR: 8260165: CSSFilterTest.testCSSFilterRendering system test fails

2021-02-24 Thread Kevin Rushforth
On Wed, 24 Feb 2021 16:10:44 GMT, Kevin Rushforth wrote: >> Issue: Initial layout delay was removed and layout() is called from >> layoutTimer instead of WebPage::prePaint(). >> >> Fix: Re-introduce the initial layout delay. > > Looks good. Tested on all three platforms. @johanvos @guruhb can

Re: RFR: 8260165: CSSFilterTest.testCSSFilterRendering system test fails

2021-02-24 Thread Kevin Rushforth
On Mon, 22 Feb 2021 11:55:00 GMT, Arun Joseph wrote: > Issue: Initial layout delay was removed and layout() is called from > layoutTimer instead of WebPage::prePaint(). > > Fix: Re-introduce the initial layout delay. Looks good. Tested on all three platforms. - Marked as

Re: RFR: 8260165: CSSFilterTest.testCSSFilterRendering system test fails

2021-02-24 Thread Arun Joseph
On Wed, 24 Feb 2021 13:01:50 GMT, Kevin Rushforth wrote: >> Issue: Initial layout delay was removed and layout() is called from >> layoutTimer instead of WebPage::prePaint(). >> >> Fix: Re-introduce the initial layout delay. > > This seems a safe enough fix to restore the layout delay that was

Re: RFR: 8260165: CSSFilterTest.testCSSFilterRendering system test fails

2021-02-24 Thread Kevin Rushforth
On Mon, 22 Feb 2021 11:55:00 GMT, Arun Joseph wrote: > Issue: Initial layout delay was removed and layout() is called from > layoutTimer instead of WebPage::prePaint(). > > Fix: Re-introduce the initial layout delay. This seems a safe enough fix to restore the layout delay that was removed in

RFR: 8260165: CSSFilterTest.testCSSFilterRendering system test fails

2021-02-24 Thread Arun Joseph
Issue: Initial layout delay was removed and layout() is called from layoutTimer instead of WebPage::prePaint(). Fix: Re-introduce the initial layout delay. - Commit messages: - Add comment - 8260165: CSSFilterTest.testCSSFilterRendering system test fails Changes: