Rearrange progress item operations from FileActionMgr to perform safe operations.

The call to ProgressItem::advance should now return true once and only once
with last item, so an (still unsafe) pointer check should not be needed.