Remove extraneous changes beyond avoiding the false positive warnings, since super.shutdown() is never expected to throw.

Also remove redundant unit test.

Rework of PR grpc#12705.

Author: kannanjgithub

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

@@ -73,13 +73,12 @@ public ManagedChannel shutdown() {
 
   @Override
   public ManagedChannel shutdownNow() {
-    ManagedChannel result = super.shutdownNow();
     phantom.clearSafely();
     // This dummy check prevents the JIT from collecting 'this' too early
     if (this.getClass() == null) {
       throw new AssertionError();
     }
-    return result;
+    return super.shutdownNow();
   }
 
   @VisibleForTesting

core/src/test/java/io/grpc/internal/ManagedChannelOrphanWrapperTest.java

@@ -140,31 +140,6 @@ public boolean isLoggable(LogRecord record) {
     }
   }
 
-  @Test
-  public void orphanedChannel_triggerWarningAndCoverage() {
-    ManagedChannel mc = new TestManagedChannel();
-    final ReferenceQueue<ManagedChannelOrphanWrapper> refqueue = new ReferenceQueue<>();
-    ConcurrentMap<ManagedChannelReference, ManagedChannelReference> refs = 
-        new ConcurrentHashMap<>();
-    
-    // Create the wrapper but NEVER call shutdown
-    @SuppressWarnings("UnusedVariable")
-    ManagedChannelOrphanWrapper wrapper = new ManagedChannelOrphanWrapper(mc, refqueue, refs);
-    wrapper = null; // Make it eligible for GC
-
-    // Trigger GC and clean the queue to hit the !wasShutdown branch
-    final AtomicInteger numOrphans = new AtomicInteger();
-    GcFinalization.awaitDone(new FinalizationPredicate() {
-      @Override
-      public boolean isDone() {
-        numOrphans.getAndAdd(ManagedChannelReference.cleanQueue(refqueue));
-        return numOrphans.get() > 0;
-      }
-    });
-    
-    assertEquals(1, numOrphans.get());
-  }
-
   @Test
   public void refCycleIsGCed() {
     ReferenceQueue<ManagedChannelOrphanWrapper> refqueue =