[PowerStatus] crashes Deskbar

[diver]

hrev47284.

• Install PowerStatus replicant
• Open Battery info window
• Quit replicant

[waddlesplash]

I wonder if this crash always happens, or does it only happen on systems with a battery?

[gbl08ma]

After some experimentation, it seems the crash occurs because the PowerStatusReplicant destructor stops and releases the reference of the DriverInterface while the extended battery info window is still using it.

If _Quit() is modified to always close the battery information window before removing the replicant from the deskbar, the deskbar no longer crashes under the specified conditions.

However, with this modification, PowerStatus still crashes in a different situation, presumably for the same reason. Steps to reproduce:

1. Open the PowerStatus applet and specify to Run in window

2. Drag, using the bottom-right corner, the replicant to the desktop

3. Close the PowerStatus window (not the replicant)

4. Right-click the battery icon on the replicant and select "Battery info...", leave the window open.

5. Right-click the dragger on the bottom-right of the replicant and select "Remove replicant.". The Tracker will crash.

I'm not sure how to solve this last problem as _Quit() is not called in this situation (in fact, using the "Quit" option provided by PowerStatus won't work when it is a stand-alone replicant). Perhaps some message (that is not B_QUIT_REQUESTED) will be sent to the replicant before it is closed, or the code I added to close the window can be safely moved to the destructor?

I'll submit a patch but it is certainly not the ideal solution.

[pulkomandy]

I would go with a deeper re-strucuration of PowerStatus.

The extended info window should be started in its own dedicated team, not inside Tracker's or DeskBar's one. So, the replicant should open it by launching the main app. This way the replicant and the window can live independantly.

[Janus]

Fixed in hrev51579.