[Deskbar] freezes

[kaoutsis]

r25184, Move Deskbar to the bottom of the screen (a la windows) * Press alt+F2 * Press any team button of the Deskbar or press leaf menu (one click is enough) * Press alt+F1 Deskbar freezes

[aldeck]

I could reproduce 100%. In deskbar/TBarWindow.cpp, the workspace change triggers a BMenuBar deletion. But the BMenuBar will block in its destructor waiting for its tracking thread to end.

See the two attached backtraces. One from the TBarWindow thread and one from the BMenuBar tracking thread.

It's not reproducible on R5. As a side note, i noticed that R5 closes any open menu on workspace change, whereas haiku keeps tracking menus on the previous workspace and the menu is still open when you come back.

I couldn't go further :) cc'ing to Stefano as he surely has an idea ;)

[jackburton]

Replying to aldeck:

I couldn't go further :) cc'ing to Stefano as he surely has an idea ;)

Yes, I have :) BMenu and BMenuBar tracking loops should store the current workspace on the first loop, and then check if the workspace changed, on every iteration. I'm afraid this will uglify the code even more.

[aldeck]

Replying to jackburton:

Yes, I have :) BMenu and BMenuBar tracking loops should store the current workspace on the first loop, and then check if the workspace changed, on every iteration. I'm afraid this will uglify the code even more.

Well the code does the job and is not that ugly :) If you find the time, i think it is worth giving a try.

Regards, Alex