It is also no longer a user visible field

# Conflicts:
#	src/MissionManager/MissionManagerTest.cc

Also additional major Mission usability changes

- override by vehicle type
- override by firmware type

- Fix: Version error message not being displayed
- Correct handling of APM and PX4 files with respect to home position
in first slot
- Emit newItemsFromVehicle signal so editor can fit map viewport on
load from vehicle

Previous order was signalling new mission items available prior to in
progress going false. This was causing MissionController to pull an
empty mission list.

Hamburger: Delete, Delete All, Show all Values

Hamburger: Delete, Delete All, Show all Values

There are many ways of iterating over lists in C++/Qt.
In the specific case of QStrings, it is known [1] that using
foreach with const references avoids the creation of new
QString objects in each iteration, reducing the time consumed
by the loops.

[1] https://blog.qt.io/blog/2009/01/23/iterating-efficiently/



Signed-off-by: Anselmo L. S. Melo <anselmo.melo@intel.com>