The QString behaves faster if used with const - reference on
a QForeach.

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>

if an unknown string was passed to createInnerDockidget,
the widget would not be created but a reference to the null
pointer would still be saved on the avaliable dock widgets,
and thus any code that foreached the list accessing methods
could led to a crash.

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>

Since it's just a state-toogler, use the return value of the
method to set the values directly.

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>

This is in preparation for full Qml main window

- Delete all old map code

- select active vehicle
- arm/disarm
- select flight mode

Also:
- Remove unused code

Too complicated supporting both modes as editor becomes more complex

These use OpenGL implementation which prevented moving to ANGLE
support. Since they are rarely used they are being remove instead of
being upgrade.

These use OpenGL implementation which prevented moving to ANGLE
support. Since they are rarely used they are being remove instead of
being upgrade.

Plus lots of other cleanup

- MultiVehicleManager is a Qml friendly replacement for UASManager
- Vehicle is a replacement for both UAS and AutoPilotPlugin