Fix up if statement curly braces issues.

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

Index: components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java
diff --git a/components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java b/components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java
index 9ec0f9efd78fd3d8783e2fcec3c1de41cf0c6b2c..6372a600753e50136d65a81e65a09c236d596761 100644
--- a/components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java
+++ b/components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java
@@ -686,8 +686,7 @@ public class ChromiumUrlRequest implements HttpUrlRequest {
     @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();