okhttp: Remove wrong status description

ejona86 · ejona86 · commit 5b7a21ab037d · 2016-08-01T10:43:21.000-07:00
The != should have been ==. However, it is provable that the exception
won't be null, but we want to make that fact obvious when auditing. So
we just fail if the exception is ever null.
diff --git a/okhttp/src/main/java/io/grpc/okhttp/OkHttpClientTransport.java b/okhttp/src/main/java/io/grpc/okhttp/OkHttpClientTransport.java
@@ -548,10 +548,8 @@ int getPendingStreamSize() {
    * Finish all active streams due to an IOException, then close the transport.
    */
   void onException(Throwable failureCause) {
+    Preconditions.checkNotNull(failureCause, "failureCause");
     Status status = Status.UNAVAILABLE.withCause(failureCause);
-    if (failureCause != null) {
-      status = status.augmentDescription("No provided cause");
-    }
     startGoAway(0, ErrorCode.INTERNAL_ERROR, status);
   }
 
