Bug 286244

Description Markus Keller 2009-08-11 10:03:07 EDT

HEAD

Run the snippet below on GTK and on WinXP. When the window is open, click into your workbench window. Wait for 10 seconds (the window activates itself), then click the close box of the window.

Sequence on WinXP:             Sequence on GTK:
------------------             ----------------
shellActivated()               shellActivated()
shellDeactivated()             shellDeactivated()
forcing active                 forcing active
shellActivated()           +----
shell is active: true      |   shell is active: true
forced active              |   forced active
                           +-> shellActivated()


=> On GTK, the shellActivated event is only sent out long after the shell became active (when the display loop is run the next time).

Workaround is to manually run the display loop. If you e.g. add

	long start = System.currentTimeMillis();
	while (start + 2000 > System.currentTimeMillis()) {
		while (display.readAndDispatch())
			;
	}

after shell.forceActive();, then the sequence becomes like the one on Windows.

This burned us in an automated test that failed as soon as the target workbench lost focus. I've worked around the problem for now.


package org.eclipse.swt.snippets;

import org.eclipse.swt.events.*;
import org.eclipse.swt.widgets.*;

public class ShellActivate {

    public static void main(String[] args) {
        final Display display= new Display();
        final Shell shell= new Shell(display);
        shell.addShellListener(new ShellAdapter() {
            public void shellDeactivated(ShellEvent e) {
                System.out.println("shellDeactivated()");
                display.timerExec(2000, new Runnable() {
                    public void run() {
                        System.out.println("forcing active");
                        shell.forceActive();
                        System.out.println("shell is active: "
                                + (display.getActiveShell() == shell));
                        System.out.println("forced active");
                        try {
                            Thread.sleep(5000);
                        } catch (InterruptedException e) {
                            e.printStackTrace();
                        }
                    }
                });
            }

            public void shellActivated(ShellEvent e) {
                System.out.println("shellActivated()");
            }
        });

        shell.setSize(160, 100);
        shell.open();
        while (!shell.isDisposed()) {
            if (!display.readAndDispatch())
                display.sleep();
        }
        display.dispose();
    }
}