From fbd60a46ec43bbf3d31789a851476ec4cd74d8ae Mon Sep 17 00:00:00 2001
From: Jonathan Nieder <jrn@google.com>
Date: Sat, 19 Jan 2019 19:32:17 -0800
Subject: [PATCH] Remove @VisibleForTesting annotation on
 ChangeKindCacheImpl#NoCache

It is not used in any tests.  It is useful when using Gerrit as a
library, so let's make this helper an ordinary public class.

Noticed via an error-prone warning about a Google-internal caller.

Change-Id: I6fe9f8a792b7fa9328f95aa97893bcec78db367a
---
 java/com/google/gerrit/server/change/ChangeKindCacheImpl.java | 1 -
 1 file changed, 1 deletion(-)

diff --git a/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java b/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java
index 27def1ceec..170223dae4 100644
--- a/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java
+++ b/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java
@@ -79,7 +79,6 @@ public class ChangeKindCacheImpl implements ChangeKindCache {
     };
   }
 
-  @VisibleForTesting
   public static class NoCache implements ChangeKindCache {
     private final boolean useRecursiveMerge;
     private final ChangeData.Factory changeDataFactory;