android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java - Issue 672533002: Enable separatorWrap module in CheckStyle and fix all the issues.

Unified Diff: android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java

Issue 672533002: Enable separatorWrap module in CheckStyle and fix all the issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Undo mojo changes 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 | « android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java ('k') | chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java

diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java

index 4dee1cfda46204adfb3730e3695002333841bb2d..2c72619ce57c275cf1cf045959c387a274c152d9 100644

--- a/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java

+++ b/android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java

@@ -116,10 +116,10 @@ public class CookieManagerTest extends AwTestBase {

JSUtils.executeJavaScriptAndWaitForResult(

this, mAwContents,

mContentsClient.getOnEvaluateJavaScriptResultHelper(),

- "var expirationDate = new Date();" +

- "expirationDate.setDate(expirationDate.getDate() + 5);" +

- "document.cookie='" + name + "=" + value +

- "; expires=' + expirationDate.toUTCString();");

+ "var expirationDate = new Date();"

+ + "expirationDate.setDate(expirationDate.getDate() + 5);"

+ + "document.cookie='" + name + "=" + value

+ + "; expires=' + expirationDate.toUTCString();");

}

@MediumTest

@@ -425,8 +425,8 @@ public class CookieManagerTest extends AwTestBase {

* @return the url which gets the response

private String makeScriptLinkUrl(TestWebServer webServer, String path, String url) {

- String responseStr = "<html><head><title>Content!</title></head>" +

- "<body><script src=" + url + "></script></body></html>";

+ String responseStr = "<html><head><title>Content!</title></head>"

+ + "<body><script src=" + url + "></script></body></html>";

return webServer.setResponse(path, responseStr, null);

}

@@ -436,8 +436,8 @@ public class CookieManagerTest extends AwTestBase {

TestWebServer webServer = TestWebServer.start();

try {

// This test again uses 127.0.0.1/localhost trick to simulate a third party.

- ThirdPartyCookiesTestHelper thirdParty

- = new ThirdPartyCookiesTestHelper(webServer);

+ ThirdPartyCookiesTestHelper thirdParty =

+ new ThirdPartyCookiesTestHelper(webServer);

mCookieManager.setAcceptCookie(true);

assertTrue(mCookieManager.acceptCookie());

@@ -597,8 +597,8 @@ public class CookieManagerTest extends AwTestBase {

* @return the url which gets the response

private String makeIframeUrl(TestWebServer webServer, String path, String url) {

- String responseStr = "<html><head><title>Content!</title></head>" +

- "<body><iframe src=" + url + "></iframe></body></html>";

+ String responseStr = "<html><head><title>Content!</title></head>"

+ + "<body><iframe src=" + url + "></iframe></body></html>";

return webServer.setResponse(path, responseStr, null);

}

@@ -612,8 +612,8 @@ public class CookieManagerTest extends AwTestBase {

private String makeCookieScriptUrl(TestWebServer webServer, String path, String key,

String value) {

- String response = "<html><head></head><body>" +

- "<script>document.cookie = \"" + key + "=" + value + "\";</script></body></html>";

+ String response = "<html><head></head><body>"

+ + "<script>document.cookie = \"" + key + "=" + value + "\";</script></body></html>";

return webServer.setResponse(path, response, null);

}