<AWT Dev> RFR: 8040630 Popup menus and tooltips flicker with previous popup contents when first shown

[Alexander Zuev]

Hi Sergey,

   as a workaround resetting the internal windows state will do. 
Question is - do we use OpenGL framework
anywhere aside of OSX? I doubt we do. And does new Metal framework 
exhibit this problem? I know this
question is probably not for you specifically, but still.

   Anyways - shouldn't we add some specific comments about what this 
code is workaround for so we don't
forget to clean it up later when the main problem is gone (one way or 
another)?

/Alex

On 03-Feb-20 5:07, Sergey Bylokhov wrote:
> Hello.
> Please review the fix for jdk/client.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8040630
> Fix: http://cr.openjdk.java.net/~serb/8040630/webrev.00
>
> This change is intended to fix the flickering of popup menus on
> macOS. The root cause of the problem is that the native windows
> for popup menus are cached internally. So when we show such a
> cached window to the user he will see old content till the moment
> we draw the new one.
> As a fix we could
>  - Drop the internal state(surface) of the window which contains the
>    old content, as suggested in the current fix we set its size to 1x1 
> pixels.
>  - Clean the old content by something like graphics.clearRect(), but
>    since the draw operation is asynchronous it is possible it will
>    not be completed when the user will see the popup.
>  - Drop the caching completely, I am not sure that we really needed it,
>    I have checked the performance PopupFactory.getPopup() when the
>    cache is enabled/disabled and did not found a big difference. But
>    since I plan to backport this to jdk11, I would like to minimize the
>    fix, and drop the cache sometime later.
>
> Notes:
>  - The easiest way to reproduce the bug is to run SwingSet2, and
>    quickly open/reopen the "File" and "Look and feel" menu.
>  - The bug is not really macOS specific, but java2D OGL pipeline"
>    specific where the actual draw operations are asynchronous.
>
>