blackbox: move _openlogfile to a separate method
This removes several `stat` syscalls for accessing `_bbvfs` and makes `ui`
object cleaner.
Differential Revision: https://phab.mercurial-scm.org/D769
#require docutils gettext
$ $TESTDIR/check-gendoc zh_TW
checking for parse errors