Fix Java indentation issues.

Fix Java indentation issues.

Fix indentation issues to allow enabling Checkstyle indentation rule.

BUG=318404
Committed: https://crrev.com/cd44405f000d57d6481eaf33e3980747db7f771a
Cr-Commit-Position: refs/heads/master@{#300162}

[aurimas (slooooooooow), 2014-10-16 19:14:52]

aurimas@chromium.org changed reviewers:
+ fgorski@chromium.org, mef@chromium.org, mnaganov@chromium.org,
nyquist@chromium.org, tedchoc@chromium.org

[aurimas (slooooooooow), 2014-10-16 19:14:53]

tedchoc@chromium.org: Please review changes in 
web_contents_delegate_android/

fgorski@chromium.org: Please review changes in 
gcm_driver/

mnaganov@chromium.org: Please review changes in 
devtools/

nyquist@chromium.org: Please review changes in 
dom_distiller/ and invalidation/

mef@chromium.org: Please review changes in 
cronet/

[fgorski, 2014-10-16 19:18:20]

lgtm

[mef, 2014-10-16 19:34:39]

lgtm, with questions. :)

Does it make sense to limit line length to 80 to avoid line wrapping in
Rietveld?

https://codereview.chromium.org/662603004/diff/1/components/cronet/android/te...
File
components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java
(right):

https://codereview.chromium.org/662603004/diff/1/components/cronet/android/te...
components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java:84:
"<p>Test page loaded.</p>\n</head>\n</html>\n",
should this line be indented further?

[Ted C, 2014-10-16 19:49:05]

lgtm

[aurimas (slooooooooow), 2014-10-16 20:28:21]

Java style sets line limit to 100.

You can adjust retvield:
https://screenshot.googleplex.com/gUu51Zrvs5

https://chromiumcodereview.appspot.com/662603004/diff/1/components/cronet/and...
File
components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java
(right):

https://chromiumcodereview.appspot.com/662603004/diff/1/components/cronet/and...
components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java:84:
"<p>Test page loaded.</p>\n</head>\n</html>\n",
On 2014/10/16 19:34:38, mef wrote:
> should this line be indented further?

Why should it be intended further? It is +8 spaces from the parent line
assertEquals(.

[aurimas (slooooooooow), 2014-10-16 20:28:22]

Java style sets line limit to 100.

You can adjust retvield:
https://screenshot.googleplex.com/gUu51Zrvs5

https://chromiumcodereview.appspot.com/662603004/diff/1/components/cronet/and...
File
components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java
(right):

https://chromiumcodereview.appspot.com/662603004/diff/1/components/cronet/and...
components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java:84:
"<p>Test page loaded.</p>\n</head>\n</html>\n",
On 2014/10/16 19:34:38, mef wrote:
> should this line be indented further?

Why should it be intended further? It is +8 spaces from the parent line
assertEquals(.

[mnaganov (inactive), 2014-10-17 07:40:21]

devtools_bridge/ changes LGTM

Thanks for fixing!

[aurimas (slooooooooow), 2014-10-17 15:01:07]

Nyquist: ping?

[nyquist, 2014-10-17 20:38:05]

dom_distiller/ and invalidation/ lgtm

[aurimas (slooooooooow), 2014-10-17 20:38:26]

The CQ bit was checked by aurimas@chromium.org

[commit-bot: I haz the power, 2014-10-17 20:39:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/662603004/1

[commit-bot: I haz the power, 2014-10-17 21:12:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-17 21:15:42]

Patchset 1 (id:??) landed as
https://crrev.com/cd44405f000d57d6481eaf33e3980747db7f771a
Cr-Commit-Position: refs/heads/master@{#300162}