Activity runner is now useless:
m_activities is always nullptr in ActivityRunner::run because ActivityRunner::down is called after ActivityRunner::match which makes activity changing impossible
|Unit Tests Skipped|
Urgh. I suspect this was a hidden bug from when runnermanager never emitted teardown.
Given how runnermanager currently seems to work, this patch does make sense.
you don't need this new code, m_activites has this as a parent, we can just delete the code in down()