content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java - Issue 744453002: Fix a bunch of Java Checkstyle issues.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java

Issue 744453002: Fix a bunch of Java Checkstyle issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: NeedsBraces to info Created 6 years, 1 month 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 | « content/public/android/javatests/src/org/chromium/content/browser/TestsJavaScriptEvalTest.java ('k') | content/public/android/javatests/src/org/chromium/content/browser/input/SelectPopupTest.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java

diff --git a/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java b/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java

index 30b879bd00bfdff39c57276b521fa91fd98a5d0e..1d84ac65dca28fcee245d7956d42b561ded01f35 100644

--- a/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java

+++ b/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java

@@ -39,14 +39,14 @@ import java.util.concurrent.Callable;

public class ImeTest extends ContentShellTestBase {

private static final String DATA_URL = UrlUtils.encodeHtmlDataUri(

- "<html><head><meta name=\"viewport\"" +

- "content=\"width=device-width, initial-scale=2.0, maximum-scale=2.0\" /></head>" +

- "<body><form action=\"about:blank\">" +

- "<input id=\"input_text\" type=\"text\" /><br/>" +

- "<input id=\"input_radio\" type=\"radio\" style=\"width:50px;height:50px\" />" +

- "<br/><textarea id=\"textarea\" rows=\"4\" cols=\"20\"></textarea>" +

- "<br/><p><span id=\"plain_text\">This is Plain Text One</span></p>" +

- "</form></body></html>");

+ "<html><head><meta name=\"viewport\""

+ + "content=\"width=device-width, initial-scale=2.0, maximum-scale=2.0\" /></head>"

+ + "<body><form action=\"about:blank\">"

+ + "<input id=\"input_text\" type=\"text\" /><br/>"

+ + "<input id=\"input_radio\" type=\"radio\" style=\"width:50px;height:50px\" />"

+ + "<br/><textarea id=\"textarea\" rows=\"4\" cols=\"20\"></textarea>"

+ + "<br/><p><span id=\"plain_text\">This is Plain Text One</span></p>"

+ + "</form></body></html>");

private TestAdapterInputConnection mConnection;

private ImeAdapter mImeAdapter;

@@ -731,8 +731,8 @@ public class ImeTest extends ContentShellTestBase {

assertTrue(CriteriaHelper.pollForCriteria(new Criteria() {

@Override

public boolean isSatisfied() {

- return show == getImeAdapter().mIsShowWithoutHideOutstanding &&

- (!show || getAdapterInputConnection() != null);

+ return show == getImeAdapter().mIsShowWithoutHideOutstanding

+ && (!show || getAdapterInputConnection() != null);

}

}));

}