Commit dfe766bf authored by Nate Fischer's avatar Nate Fischer Committed by Commit Bot

AW: re-enable flaky test

No change to logic. This re-enables a test which was marked as flaky.
This passed 1000 local runs, so it appears it's no longer flaky.

Bug: 984705
Test: run_webview_instrumentation_test_apk --num_retries=0 \
Test: --break-on-failure --repeat=1000 \
Test: -f SafeBrowsingTest#testSafeBrowsingProceedThroughInterstitialForSubresource
Change-Id: I48bb6dbbeb1fa96f62d43da74de2a34255f2b3d5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911150
Auto-Submit: Nate Fischer <ntfschr@chromium.org>
Commit-Queue: Shimi Zhang <ctzsm@chromium.org>
Reviewed-by: default avatarShimi Zhang <ctzsm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#715445}
parent 47692796
...@@ -47,7 +47,6 @@ import org.chromium.base.task.PostTask; ...@@ -47,7 +47,6 @@ import org.chromium.base.task.PostTask;
import org.chromium.base.test.util.CallbackHelper; import org.chromium.base.test.util.CallbackHelper;
import org.chromium.base.test.util.CommandLineFlags; import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.FlakyTest;
import org.chromium.base.test.util.InMemorySharedPreferences; import org.chromium.base.test.util.InMemorySharedPreferences;
import org.chromium.components.safe_browsing.SafeBrowsingApiBridge; import org.chromium.components.safe_browsing.SafeBrowsingApiBridge;
import org.chromium.components.safe_browsing.SafeBrowsingApiHandler; import org.chromium.components.safe_browsing.SafeBrowsingApiHandler;
...@@ -658,7 +657,6 @@ public class SafeBrowsingTest { ...@@ -658,7 +657,6 @@ public class SafeBrowsingTest {
} }
@Test @Test
@FlakyTest(message = "crbug.com/984705")
@SmallTest @SmallTest
@Feature({"AndroidWebView"}) @Feature({"AndroidWebView"})
public void testSafeBrowsingProceedThroughInterstitialForSubresource() throws Throwable { public void testSafeBrowsingProceedThroughInterstitialForSubresource() throws Throwable {
......
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