Review Request: Workaround Taskbar ghost entries bug

[Craig Drummond]

I don’t know if its related, but whilst working on IconTasks I noticed that 
items set to be deleted via deleteLater (in TaskGroupItem::itemRemoved and 
TaskManager::TaskItem) were not being deleted until after the app 
(plasmoidviewer / plasma-desktop) had terminated. To trace this I put debug 
when deleteLater was called, and then in the destructor. I have checked with 
the standard taskbar applet from KDE/4.7, and it has the same issue.

I worked-around this by using a single-shot QTimer to call deleteLater() - but 
this seems a little hacky to me.

Any ideas why the destructor is not being called?

Craig.

p.s. I also noticed a bug with AbstractTaskItem::setGeometry(). If the passed 
in geometry is the same as the current geometry, the layout animation and 
update-geometry timers need to be stopped. Otherwise a gap can appear when a 
group of items is closed all at once.