Commit a5ef8671 authored by Donn Denman's avatar Donn Denman Committed by Commit Bot

[TTS] Reenable CSMTest suite with longer timeout.

Speculatively re-enables the ContextualSearchManagerTest suite with
the hope that the mysterious underlying flakiness issue has gone
away, or that the longer timeout in this version of the code will
help.

Original disabling CL: https://crrev.com/c/2088007

TBR=twellington@chromium.org

BUG=1058362

Change-Id: I02ea4f209b86b730cc6f40622839ffb12f3753c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094245
Commit-Queue: Donn Denman <donnd@chromium.org>
Reviewed-by: default avatarDonn Denman <donnd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748287}
parent 5fefa0cd
...@@ -125,11 +125,6 @@ import java.util.concurrent.TimeoutException; ...@@ -125,11 +125,6 @@ import java.util.concurrent.TimeoutException;
"disable-features=" + ChromeFeatureList.CONTEXTUAL_SEARCH_ML_TAP_SUPPRESSION}) "disable-features=" + ChromeFeatureList.CONTEXTUAL_SEARCH_ML_TAP_SUPPRESSION})
@Restriction(RESTRICTION_TYPE_NON_LOW_END_DEVICE) @Restriction(RESTRICTION_TYPE_NON_LOW_END_DEVICE)
@RetryOnFailure @RetryOnFailure
//
// NOTE -- THIS WHOLE TEST CLASS IS TEMPORARILY DISABLED:
//
@DisabledTest(message = "See https://crbug.com/1058362")
//
public class ContextualSearchManagerTest { public class ContextualSearchManagerTest {
@Rule @Rule
public ChromeActivityTestRule<ChromeActivity> mActivityTestRule = public ChromeActivityTestRule<ChromeActivity> mActivityTestRule =
...@@ -147,7 +142,7 @@ public class ContextualSearchManagerTest { ...@@ -147,7 +142,7 @@ public class ContextualSearchManagerTest {
private static final String LOW_PRIORITY_INVALID_SEARCH_ENDPOINT = "/s/invalid"; private static final String LOW_PRIORITY_INVALID_SEARCH_ENDPOINT = "/s/invalid";
private static final String CONTEXTUAL_SEARCH_PREFETCH_PARAM = "&pf=c"; private static final String CONTEXTUAL_SEARCH_PREFETCH_PARAM = "&pf=c";
// The number of ms to delay startup for all tests. // The number of ms to delay startup for all tests.
private static final int ACTIVITY_STARTUP_DELAY_MS = 1000; private static final int ACTIVITY_STARTUP_DELAY_MS = 5000; // TODO(donnd): reduce!
private ActivityMonitor mActivityMonitor; private ActivityMonitor mActivityMonitor;
private ContextualSearchFakeServer mFakeServer; private ContextualSearchFakeServer mFakeServer;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment