Expose _show_tab as show_tab from now on

Found sufficient (even if not always hack-free) distribution throughout
our codebase to justify this.
Remove superfluous FightingShipOverviewTab icon definition

The same as ShipOverviewTab it inherits from.
Rename method for more clarity
Add assertion for debugging purposes