core: deprecate LoadBalancer.Helper#getNameResolverFactory

core/src/main/java/io/grpc/LoadBalancer.java

@@ -638,7 +638,11 @@ public ScheduledExecutorService getScheduledExecutorService() {
      * Returns the NameResolver of the channel.
      *
      * @since 1.2.0
+     *
+     * @deprecated this method will be deleted in a future release.  If you think it shouldn't be
+     *     deleted, please file an issue on <a href="https://github.com/grpc/grpc-java">github</a>.
      */
+    @Deprecated
     public abstract NameResolver.Factory getNameResolverFactory();
 
     /**

core/src/main/java/io/grpc/internal/ManagedChannelImpl.java

@@ -1256,6 +1256,7 @@ public String getAuthority() {
       return ManagedChannelImpl.this.authority();
     }
 
+    @Deprecated
     @Override
     public NameResolver.Factory getNameResolverFactory() {
       return nameResolverFactory;

core/src/main/java/io/grpc/util/ForwardingLoadBalancerHelper.java

@@ -76,6 +76,7 @@ public void runSerialized(Runnable task) {
     delegate().runSerialized(task);
   }
 
+  @Deprecated
   @Override
   public NameResolver.Factory getNameResolverFactory() {
     return delegate().getNameResolverFactory();

core/src/test/java/io/grpc/LoadBalancerTest.java

@@ -211,7 +211,9 @@ public void updateBalancingState(
       return null;
     }
 
-    @Override public NameResolver.Factory getNameResolverFactory() {
+    @Deprecated
+    @Override
+    public NameResolver.Factory getNameResolverFactory() {
       return null;
     }
 

services/src/test/java/io/grpc/services/HealthCheckingLoadBalancerFactoryTest.java

@@ -1190,6 +1190,7 @@ public ScheduledExecutorService getScheduledExecutorService() {
       return clock.getScheduledExecutorService();
     }
 
+    @Deprecated
     @Override
     public NameResolver.Factory getNameResolverFactory() {
       throw new AssertionError("Should not be called");