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

AW: disable testOnSafeBrowsingHit CTS test

The WebViewClientTest#testOnSafeBrowsingHit() is flaky due to a subtle
timing change when we switched to Network Service. This disables the
test for now, while we look for a fix.

This test was added in OMR1 (which we don't run CTS tests for in
chromium waterfall), so we don't need to do anything for earlier API
levels. The flaky part of the test was split out into
testOnSafeBrowsingHitBackToSafety in Q, which we're already skipping in
the JSON config.

Bug: 1006953
Test: android_webview/tools/run_cts.py (on P device)
Change-Id: I2c3e197484b1678722573ee052aed83150148417
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975156
Auto-Submit: Nate Fischer <ntfschr@chromium.org>
Commit-Queue: Andrew Luo <aluo@chromium.org>
Reviewed-by: default avatarAndrew Luo <aluo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#726610}
parent 8e917b47
...@@ -245,7 +245,13 @@ ...@@ -245,7 +245,13 @@
}, },
"test_runs": [ "test_runs": [
{ {
"apk": "android-cts/testcases/CtsWebkitTestCases.apk" "apk": "android-cts/testcases/CtsWebkitTestCases.apk",
"excludes": [
{
"match": "android.webkit.cts.WebViewClientTest#testOnSafeBrowsingHit",
"_bug_id": "crbug.com/1006953"
}
]
}, },
{ {
"apk": "android-cts/testcases/CtsWebViewStartupApp.apk" "apk": "android-cts/testcases/CtsWebViewStartupApp.apk"
......
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