diff --git a/trac/versioncontrol/svn_authz.py b/trac/versioncontrol/svn_authz.py
--- a/trac/versioncontrol/svn_authz.py
+++ b/trac/versioncontrol/svn_authz.py
@@ -139,7 +139,7 @@
             usernames = ('$anonymous', '*')
         else:
             usernames = (username, '$authenticated', '*')
-        if action == 'FILE_VIEW' or action == 'BROWSER_VIEW':
+        if action in ('BROWSER_VIEW', 'FILE_VIEW', 'LOG_VIEW'):
             authz, users = self._get_authz_info()
             if authz is None:
                 return False
diff --git a/trac/versioncontrol/tests/svn_authz.py b/trac/versioncontrol/tests/svn_authz.py
--- a/trac/versioncontrol/tests/svn_authz.py
+++ b/trac/versioncontrol/tests/svn_authz.py
@@ -200,8 +200,9 @@
         """
         resource = Resource('source', path,
                             parent=Resource('repository', reponame))
-        check = self.policy.check_permission('FILE_VIEW', user, resource, None)
-        self.assertEqual(result, check)
+        for perm in ('BROWSER_VIEW', 'FILE_VIEW', 'LOG_VIEW'):
+            check = self.policy.check_permission(perm, user, resource, None)
+            self.assertEqual(result, check)
         
     def test_default_permission(self):
         # By default, permissions are undecided