Cleanup failure case for moderated topic view

TopicBlock::loadTopicTitle was only returning the revision instance
when the user had 'view' accessto the revision, which is reasonable
but not everwhere was checking this return value appropriatly.

Additionally there are cases when we only need the lower level of 'history'
access and not the full-content 'view' access.  To accomidate this
A new parameter $action was added to TopicBlock::loadTopicTitle
to specify the level of access needed as well as reviewing and cleaning up
return value usage for all places that use the method.

Bug: 67595
Change-Id: I12f83df3a1f3d49fd5abd9bc9de8d0a4ae4ed29c
Merge "Cleanup failure case for moderated topic view"