Externalize does not repaint frame

[Sciss]

I am seeing this problem both with 1.1.1 and latest snapshot and across different JVM (OpenJDK 6, OpenJDK 7, Oracle JDK 8) and look-and-feels (Motif, Metal, Native, ...), so assume it's a bug in docking-frames. I have a very simple layout with one single and one multi dock. If I try to externalise either frame, I just get a blank rectangle, the frame is not painted (neither border decoration nor content). I have to explicitly deactivate and reactive the window and then move it to have it fully painted.

[Benoker]

Swing is broken in Linux since about version 1.6. Not just Docking-Frames has repaint issues, any application that uses more than one frame has issues (we have a nice Swing application at my workplace that does not work properly on many different Linux computers).

[Sciss]

This is simply not true. I am using a great number of Swing based multi-window applications. It may be that there are bugs, but they are usually not consistent across OpenJDK 6, OpenJDK 7 and Oracle JDK 8. I am sure there are possible work-arounds. For example, is it possible to turn off the animation that creates the externalised frame?

[Sciss]

Making the dialog displayable by calling pack in ScreenDockDialog.java mostly fixes this problem:

        if( undecorated ){
            dialog.setUndecorated( true );
            dialog.getRootPane().setWindowDecorationStyle( JRootPane.NONE );
            dialog.pack();  // N.B. required for Linux
        }

[Sciss]

The second change necessary is calling updateBorder in the Listener in AbstractScreenDockWindow, e.g. by adding a constructor:

        private Listener() {
            updateBorder();
        }

[Benoker]

pack may be a bit dangerous, that could have unwanted side effects (like the size of the dialog not being the expected size). I'll try out your fixes, I'm really curious to see if these changes help on my system too (due to having a new computer I first need to reinstall my Ubuntu... could take a while).

[Sciss]

I double checked that the size is not initialised at that moment. If there is doubt, one may just paste the contents of pack minus setting the size:

  Container parent = this.parent;
  if (parent != null && parent.getPeer() == null) {
    parent.addNotify();
  }
  if (peer == null) {
    addNotify();
  }