Revert "HttpClient: attempt to avoid content length header parsing if we have a body content length we can use"

This reverts commit f730779.

We always want to prefer the header supplied value if it's available.  This logic is just simpler to understand and likely safer.

Author: jentfoo

client/src/main/java/org/threadly/litesockets/client/http/HTTPClient.java

@@ -594,15 +594,11 @@ public HTTPResponseCode getResponseCode() {
     }
 
     public long getContentLength() {
-      long result = body.remaining();
-      if (result > 0) {
-        return result;
-      }
-      result = hr.getHeaders().getContentLength();
-      if (result > 0) {
+      long result = hr.getHeaders().getContentLength();
+      if (result >= 0) {
         return result;
       } else {
-        return 0;
+        return body.remaining();
       }
     }
 

protocol/src/main/java/org/threadly/litesockets/protocols/http/response/HTTPResponseProcessor.java

@@ -138,8 +138,7 @@ public void connectionClosed() {
           reset(new HTTPParsingException("Did not complete chunked encoding!"));
         }
       } else {
-        if(response.getHeaders().getContentLength() > 0 && 
-           response.getHeaders().getContentLength() != this.currentBodySize) {
+        if(response.getHeaders().getContentLength() > 0 && response.getHeaders().getContentLength() != this.currentBodySize) {
           reset(new HTTPParsingException("Did not get complete body!"));
         } else {
           reset(null);