Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2530)

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

Issue 98853009: Fixing Java style issues in android_webview. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Bo's nits Created 7 years 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
Index: android_webview/javatests/src/org/chromium/android_webview/test/LoadUrlTest.java
diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/LoadUrlTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/LoadUrlTest.java
index 76d226594850b9c2ac0320df2d6a5a0944966d1e..f283b2dc1f54a176abb65a5a9f8dcc995cbb49a7 100644
--- a/android_webview/javatests/src/org/chromium/android_webview/test/LoadUrlTest.java
+++ b/android_webview/javatests/src/org/chromium/android_webview/test/LoadUrlTest.java
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright 2012 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.
@@ -110,8 +110,9 @@ public class LoadUrlTest extends AwTestBase {
String[] headerNames = {"X-ExtraHeaders1", "x-extraHeaders2"};
String[] headerValues = {"extra-header-data1", "EXTRA-HEADER-DATA2"};
Map<String, String> extraHeaders = new HashMap<String, String>();
- for (int i = 0; i < headerNames.length; ++i)
- extraHeaders.put(headerNames[i], headerValues[i]);
+ for (int i = 0; i < headerNames.length; ++i) {
+ extraHeaders.put(headerNames[i], headerValues[i]);
+ }
loadUrlWithExtraHeadersSync(awContents,
contentsClient.getOnPageFinishedHelper(),
@@ -120,12 +121,12 @@ public class LoadUrlTest extends AwTestBase {
HttpRequest request = webServer.getLastRequest(path);
for (int i = 0; i < headerNames.length; ++i) {
- Header[] matchingHeaders = request.getHeaders(headerNames[i]);
- assertEquals(1, matchingHeaders.length);
+ Header[] matchingHeaders = request.getHeaders(headerNames[i]);
+ assertEquals(1, matchingHeaders.length);
- Header header = matchingHeaders[0];
- assertEquals(headerNames[i].toLowerCase(), header.getName());
- assertEquals(headerValues[i], header.getValue());
+ Header header = matchingHeaders[0];
+ assertEquals(headerNames[i].toLowerCase(), header.getName());
+ assertEquals(headerValues[i], header.getValue());
}
} finally {
if (webServer != null) webServer.shutdown();

Powered by Google App Engine
This is Rietveld 408576698