Fix up if statement curly braces issues.

components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.net;
 
 import android.util.Log;
 
 import org.apache.http.conn.ConnectTimeoutException;
 import org.chromium.base.CalledByNative;
@@ skipping 668 matching lines @@
     /**
      * Reads a sequence of bytes from upload channel into the given buffer.
      * @param dest The buffer into which bytes are to be transferred.
      * @return Returns number of bytes read (could be 0) or -1 and closes
      * the channel if error occured.
      */
     @SuppressWarnings("unused")
     @CalledByNative
     private int readFromUploadChannel(ByteBuffer dest) {
         try {
-            if (mUploadChannel == null || !mUploadChannel.isOpen())
-                return -1;

[mef, 2015/03/13 14:39:12]

Just to make sure I understand it correctly:

- Is 'if' statement expected to be one-line if it fits, or is it up to coder?
- Unless it fits on one line are curly braces always required, including simple
one-liners? 

[Ted C, 2015/03/13 16:33:20]

Up to the coder and based on readability.

But if you have something like:

if (<short boolean) return;

Then I would push for it being one line despite objections from the coder.
Correct, if it doesn't all go on one line, curly braces are always required in
Java.

+            if (mUploadChannel == null || !mUploadChannel.isOpen()) return -1;
             int result = mUploadChannel.read(dest);
             if (result < 0) {
                 mUploadChannel.close();
                 return 0;
             }
             return result;
         } catch (Exception e) {
             onCalledByNativeException(e);
         }
         return -1;
@@ skipping 46 matching lines @@
     private native void nativeGetAllHeaders(long urlRequestAdapter,
             ResponseHeadersMap headers);
 
     private native String nativeGetNegotiatedProtocol(long urlRequestAdapter);
 
     // Explicit class to work around JNI-generator generics confusion.
     private static class ResponseHeadersMap extends
             HashMap<String, List<String>> {
     }
 }