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

Unified Diff: content/public/android/javatests/src/org/chromium/content/browser/ContentViewScrollingTest.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/ContentViewReadbackTest.java ('k') | content/public/android/javatests/src/org/chromium/content/browser/GestureDetectorResetTest.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/ContentViewScrollingTest.java

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

index bece9a2d95d654c370f60290c5baab77f5c05cdb..b76e81e6d98f4807b19b509890646667965a9c3c 100644

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

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

@@ -24,13 +24,12 @@ import org.chromium.content_shell_apk.ContentShellTestBase;

public class ContentViewScrollingTest extends ContentShellTestBase {

- private static final String LARGE_PAGE = UrlUtils.encodeHtmlDataUri(

- "<html><head>" +

- "<meta name=\"viewport\" content=\"width=device-width, " +

- "initial-scale=2.0, maximum-scale=2.0\" />" +

- "<style>body { width: 5000px; height: 5000px; }</style></head>" +

- "<body>Lorem ipsum dolor sit amet, consectetur adipiscing elit.</body>" +

- "</html>");

+ private static final String LARGE_PAGE = UrlUtils.encodeHtmlDataUri("<html><head>"

+ + "<meta name=\"viewport\" content=\"width=device-width, "

+ + "initial-scale=2.0, maximum-scale=2.0\" />"

+ + "<style>body { width: 5000px; height: 5000px; }</style></head>"

+ + "<body>Lorem ipsum dolor sit amet, consectetur adipiscing elit.</body>"

+ + "</html>");

/**

* InternalAccessDelegate to ensure AccessibilityEvent notifications (Eg:TYPE_VIEW_SCROLLED)

@@ -108,12 +107,12 @@ public class ContentViewScrollingTest extends ContentShellTestBase {

final int minThreshold = 5;

final int maxThreshold = 100;

- boolean xCorrect = hugLeft ?

- getContentViewCore().getNativeScrollXForTest() < minThreshold :

- getContentViewCore().getNativeScrollXForTest() > maxThreshold;

- boolean yCorrect = hugTop ?

- getContentViewCore().getNativeScrollYForTest() < minThreshold :

- getContentViewCore().getNativeScrollYForTest() > maxThreshold;

+ boolean xCorrect = hugLeft

+ ? getContentViewCore().getNativeScrollXForTest() < minThreshold

+ : getContentViewCore().getNativeScrollXForTest() > maxThreshold;

+ boolean yCorrect = hugTop

+ ? getContentViewCore().getNativeScrollYForTest() < minThreshold

+ : getContentViewCore().getNativeScrollYForTest() > maxThreshold;

return xCorrect && yCorrect;

}

}));