On Mon, 16 Aug 2021 06:59:18 GMT, Ambarish Rapte <ara...@openjdk.org> wrote:
>> That is a good question. I believe that was actually introduced because of >> JDK-8260165 (and removed in the next update by commit >> ed0baf5f23aed0d8aaa72645c8e03fde56d0f0cc) >> >> The problem with this 250ms delay is that in case the WebView is not >> visible, it's contents won't get updated until after 250ms. That is a major >> pain if the WebView is used for printing only (hence not for rendering). > > I think this change was lost during webkit update. and it caused a regression > [JDK-8269067](https://bugs.openjdk.java.net/browse/JDK-8269067) : > CSSFilterTest fails intermittently in Windows. > This change was originally included to enable CSSFilterTest. > Adding back this change sounds like a good idea to me. The original addition of this #ifdef was due to JDK-8260165. That commit introduced the problems I mentioned above (WebView took 250ms to update if not visible), so I don't think it is a good solution. I suggest we discuss that in JDK-8269067 (but not include it in the PR with the cherry-pick of the changes, as it is not in the upstream webkit)? ------------- PR: https://git.openjdk.java.net/jfx/pull/603
