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

Unified Diff: chrome/test/android/javatests/src/org/chromium/chrome/test/util/OmniboxTestUtils.java

Issue 2573263002: Catch InterruptedException in CriteriaHelper (Closed)
Patch Set: Change OverviewModeBehaviorWatcher back to try/finally Created 4 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: chrome/test/android/javatests/src/org/chromium/chrome/test/util/OmniboxTestUtils.java
diff --git a/chrome/test/android/javatests/src/org/chromium/chrome/test/util/OmniboxTestUtils.java b/chrome/test/android/javatests/src/org/chromium/chrome/test/util/OmniboxTestUtils.java
index 2911a10dd5743af5f22d9165aab793db6929108f..b2e0ab0f38681666613bb58e1d3bd494271dcf49 100644
--- a/chrome/test/android/javatests/src/org/chromium/chrome/test/util/OmniboxTestUtils.java
+++ b/chrome/test/android/javatests/src/org/chromium/chrome/test/util/OmniboxTestUtils.java
@@ -284,8 +284,7 @@ public class OmniboxTestUtils {
* @param urlBar The UrlBar whose focus is being changed.
* @param gainFocus Whether focus should be requested or cleared.
*/
- public static void toggleUrlBarFocus(final UrlBar urlBar,
- boolean gainFocus) throws InterruptedException {
+ public static void toggleUrlBarFocus(final UrlBar urlBar, boolean gainFocus) {
if (gainFocus) {
// During early startup (before completion of its first onDraw), the UrlBar
// is not focusable. Tests have to wait for that to happen before trying to focus it.
@@ -313,8 +312,7 @@ public class OmniboxTestUtils {
* @param urlBar The UrlBar whose focus is being inspected.
* @param active Whether the UrlBar is expected to have focus or not.
*/
- public static void waitForFocusAndKeyboardActive(final UrlBar urlBar, final boolean active)
- throws InterruptedException {
+ public static void waitForFocusAndKeyboardActive(final UrlBar urlBar, final boolean active) {
CriteriaHelper.pollInstrumentationThread(new Criteria() {
@Override
public boolean isSatisfied() {
@@ -329,8 +327,7 @@ public class OmniboxTestUtils {
*
* @param locationBar The LocationBar who owns the suggestions.
*/
- public static void waitForOmniboxSuggestions(final LocationBarLayout locationBar)
- throws InterruptedException {
+ public static void waitForOmniboxSuggestions(final LocationBarLayout locationBar) {
CriteriaHelper.pollUiThread(new Criteria() {
@Override
public boolean isSatisfied() {
@@ -349,8 +346,7 @@ public class OmniboxTestUtils {
* @param expectedCount The number of suggestions expected to be shown.
*/
public static void waitForOmniboxSuggestions(
- final LocationBarLayout locationBar, final int expectedCount)
- throws InterruptedException {
+ final LocationBarLayout locationBar, final int expectedCount) {
CriteriaHelper.pollUiThread(new Criteria() {
@Override
public boolean isSatisfied() {

Powered by Google App Engine
This is Rietveld 408576698