CPTabView removeTabViewItem: fails to remove tab [patch included]

Browser: Safari 4 (untested in others)
Symptom: Calling removeTabViewItem: on a CPTabView causes the associated tab view item to be removed from its list, but does not remove the graphical label (_CPTabViewLabel) associated with it.

Cause: CPTabView uses a CPTabLabelsView class which manages the graphical tabs. It is a delegate to the CPTabView, and receives a tabView:didRemoveTabViewItem: message when the CPTabView removes a tab view item. It calls indexOfTabViewItem: on the CPTabView to determine which of its CPTabLabels to remove from the superview. However, this delegate method is called after the CPTabViewItem has been removed, so the _CPTabLabelsView delegate will always fail to find the index when indexOfTabViewItem: is called. As such, the CPTabLabel is never removed from its superview.

Solution: patch attached, however by moving the delegate call to happen before the removal of the CPTabViewItem from the CPTabView's list, the name "didRemoveTabViewItem" may be better named "willRemoveTabViewItem".