navigator: skip takeoff if already above takeoff altitude

The check here makes sense, but are we sure that report_current_offboard_mission_item() is really only executed here? There is a also a logic change here - before we ran publish_mission_result(); independent of the mission type. I don't think we should introduce logic changes unless there are reasons / test results suggesting those.