Review request #0: 6863566 (Java should support the freedesktop.org startup notification specification) (original) (raw)

Anthony Petrov Anthony.Petrov at Sun.COM
Fri Jul 31 10:34:36 UTC 2009


Hi Damjan,

On 07/30/2009 02:24 PM, Anthony Petrov wrote:

Damjan, have you by the way tested the fix with a GUI Java application that does not display a top-level window, but rather creates a tray icon only? Does the notification get correctly removed from the task bar in that case?

It doesn't :-(. Then I suggest to call the removeStartupNotification() method in the handleMapNotifyEvent() instead of the xSetVisible(). In that case it should theoretically work fine with tray-icon-only applications also. Can you test it please?

By the way, since we care about displaying top-level windows only, then perhaps we can move the code to the XWindowPeer instead of the XBaseWindow?

-- best regards, Anthony



More information about the core-libs-dev mailing list