components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java - Issue 662603004: Fix Java indentation issues.

Unified Diff: components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java

Issue 662603004: Fix Java indentation issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Created 6 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« no previous file with comments | « components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.java ('k') | components/cronet/android/test/src/org/chromium/cronet_test_apk/CronetTestActivity.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java

diff --git a/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java b/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java

index 01d9736c918c83214505081f789e49ce0cdc3bb3..19ec8228eca24b3bf665518d2c5ff0e30d562476 100644

--- a/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java

+++ b/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java

@@ -51,45 +51,45 @@ public class MockUrlRequestJobTest extends CronetTestBase {

@Feature({"Cronet"})

public void testSuccessURLRequest() throws Exception {

TestHttpUrlRequestListener listener =

- createUrlRequestAndWaitForComplete(MOCK_CRONET_TEST_SUCCESS_URL);

+ createUrlRequestAndWaitForComplete(MOCK_CRONET_TEST_SUCCESS_URL);

assertEquals(MOCK_CRONET_TEST_SUCCESS_URL, listener.mUrl);

assertEquals(200, listener.mHttpStatusCode);

assertEquals("this is a text file\n",

- new String(listener.mResponseAsBytes));

+ new String(listener.mResponseAsBytes));

}

@SmallTest

@Feature({"Cronet"})

public void testRedirectURLRequest() throws Exception {

TestHttpUrlRequestListener listener =

- createUrlRequestAndWaitForComplete(MOCK_CRONET_TEST_REDIRECT_URL);

+ createUrlRequestAndWaitForComplete(MOCK_CRONET_TEST_REDIRECT_URL);

// Currently Cronet does not expose the url after redirect.

assertEquals(MOCK_CRONET_TEST_REDIRECT_URL, listener.mUrl);

assertEquals(200, listener.mHttpStatusCode);

// Expect that the request is redirected to success.txt.

assertEquals("this is a text file\n",

- new String(listener.mResponseAsBytes));

+ new String(listener.mResponseAsBytes));

}

@SmallTest

@Feature({"Cronet"})

public void testNotFoundURLRequest() throws Exception {

TestHttpUrlRequestListener listener =

- createUrlRequestAndWaitForComplete(MOCK_CRONET_TEST_NOTFOUND_URL);

+ createUrlRequestAndWaitForComplete(MOCK_CRONET_TEST_NOTFOUND_URL);

assertEquals(MOCK_CRONET_TEST_NOTFOUND_URL, listener.mUrl);

assertEquals(404, listener.mHttpStatusCode);

assertEquals(

- "<!DOCTYPE html>\n<html>\n<head>\n<title>Not found</title>\n" +

+ "<!DOCTYPE html>\n<html>\n<head>\n<title>Not found</title>\n" +

"<p>Test page loaded.</p>\n</head>\n</html>\n",

mef 2014/10/16 19:34:38 should this line be indented further?

aurimas (slooooooooow) 2014/10/16 20:28:21 Why should it be intended further? It is +8 spaces

- new String(listener.mResponseAsBytes));

+ new String(listener.mResponseAsBytes));

}

@SmallTest

@Feature({"Cronet"})

public void testFailedURLRequest() throws Exception {

TestHttpUrlRequestListener listener =

- createUrlRequestAndWaitForComplete(MOCK_CRONET_TEST_FAILED_URL);

+ createUrlRequestAndWaitForComplete(MOCK_CRONET_TEST_FAILED_URL);

assertEquals(MOCK_CRONET_TEST_FAILED_URL, listener.mUrl);

assertEquals(0, listener.mHttpStatusCode);

}