Revert back AsyncHttpClient#844 and detach channel in finishUpdate, close AsyncHttpClient#844

Author: slandelle

src/main/java/com/ning/http/client/providers/netty/future/NettyResponseFuture.java

@@ -389,6 +389,11 @@ public void attachChannel(Channel channel, boolean reuseChannel) {
         this.reuseChannel = reuseChannel;
     }
 
+    public void detachChannel() {
+        this.channel = null;
+        this.reuseChannel = false;
+    }
+
     public Channel channel() {
         return channel;
     }

src/main/java/com/ning/http/client/providers/netty/handler/HttpProtocol.java

@@ -181,13 +181,15 @@ private void addType3NTLMAuthorizationHeader(String auth, FluentCaseInsensitiveS
 
     private void finishUpdate(final NettyResponseFuture<?> future, Channel channel, boolean expectOtherChunks) throws IOException {
 
-        markAsDone(future, channel);
+        future.detachChannel();
 
         boolean keepAlive = future.isKeepAlive();
         if (expectOtherChunks && keepAlive)
             channelManager.drainChannelAndOffer(channel, future);
         else
             channelManager.tryToOfferChannelToPool(channel, keepAlive, future.getPartition());
+        
+        markAsDone(future, channel);
     }
 
     private boolean updateBodyAndInterrupt(NettyResponseFuture<?> future, AsyncHandler<?> handler, NettyResponseBodyPart bodyPart)