Destroy a removed menu item before its action.

An item reads action() while it is being torn down, so removeAction()
deleting the action first left it pointing at freed memory for the whole
widget destructor. clearActions() and clearLastSeparator() already do it
in this order.