repoview: fix conditional around unserved changesets

The conditional could lead to wrong computation since we have more unserved
changesets than just the "secret" phase.

In addition, now that we have efficient caching of phased changesets, we don't
need the conditional anymore.

mercurial/repoview.py

@@ -93,8 +93,8 b' def computeunserved(repo, visibilityexce'
     assert not repo.changelog.filteredrevs
     # fast path in simple case to avoid impact of non optimised code
     hiddens = filterrevs(repo, 'visible')
-    if phases.hassecret(repo):
-        secrets = repo._phasecache.getrevset(repo, phases.remotehiddenphases)
+    secrets = repo._phasecache.getrevset(repo, phases.remotehiddenphases)
+    if secrets:
         return frozenset(hiddens | frozenset(secrets))
     else:
         return hiddens