Refs #0

Now the new change is in XXXPresenter::execute - must change this too

This reverts commit 1925e4fc.

The xxxLoadingPresenter objects take a raw pointer to a view in their
constructor, and then delete it in their destructor. This is confusing
to the caller. Replace the raw pointer with a unique_ptr - the presenters
now take ownership of a unique_ptr and this is much clearer. Updated all
calls in VatesAPI and tests.

We transpose any MDHistoWorkspace loaded from memory if needed.

This reverts commit 72faff79.