Workbench crash after closing a plot then defining a new fit fn

Remove FrameworkManager.h from includes in any headers

It is a header that is heavy and should never be needed in another header.
It's long include chain now also has issues with older moc versions not
parsing nested namespaces

Equality for sample run

IndirectReduction GUI slowing processing on workbench when open.

NexusFileLoader class

This caused an issue with nested imports on Qt < 5.9

Fix bugs with restoring defaults on the reflectometry GUI

Merge branch 'master' of https://github.com/mantidproject/mantid into 28424_wb_brash_close_plot_then_define_new_fit_fn

# Conflicts:
#	qt/widgets/common/src/FitPropertyBrowser.cpp

Fix test_fitpropertybrowser unreliable test

Remove algorithm proxy class

Improve Status Information in Eng Diff GUI

Corrected Fermi and sample distance on MAPS for PyChop

Add mechanism to print exception in Kafka test thread

cancel algorithm when cancelling script in workbench

Then we can remove the looped remove from the notificationCentre

The fitpropertybrowser was segfaulting when creating the editors within the init.
This is because the maps containing the property managers and factories are not cleared in the browser destructor as the properties can still be used elsewhere. In testing however these are not cleared and result in pointers to the managers and factories remaining even though the manager has been deleted.
This commit removes the relevant editors from these maps when the destructor is called.