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

Issue 2590053002: Disable failing and flaky CTS tests (Closed)

Created:
4 years ago by the real yoland
Modified:
4 years ago
CC:
android-webview-reviews_chromium.org, chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Disable failing and flaky CTS tests Disable WebViewClientTest testShouldOverrideUrlLoadingOnCreateWindow, which has been flaky on L and N bot, and ServiceWorkerClientTest testServiceWorkerClientInterceptCallback, which has been failing on N bot BUG=659077, 675809 Committed: https://crrev.com/b89f0113f539b02c742c7eb1588767282dd0ddb6 Cr-Commit-Position: refs/heads/master@{#439840}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -0 lines) Patch
M android_webview/tools/cts_config/expected_failure_on_bot.json View 1 chunk +12 lines, -0 lines 0 comments Download

Messages

Total messages: 17 (10 generated)
the real yoland
4 years ago (2016-12-20 01:58:42 UTC) #2
mikecase (-- gone --)
iirc, the testServiceWorkerClientInterceptCallback is just failing every single run on N. Might want to update ...
4 years ago (2016-12-20 02:15:04 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2590053002/1
4 years ago (2016-12-20 17:17:03 UTC) #7
mikecase (-- gone --)
I think you mistakenly made the commit msg suuuuper long.
4 years ago (2016-12-20 17:23:09 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2590053002/1
4 years ago (2016-12-20 17:52:58 UTC) #12
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years ago (2016-12-20 17:58:58 UTC) #15
commit-bot: I haz the power
4 years ago (2016-12-20 18:01:56 UTC) #17
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/b89f0113f539b02c742c7eb1588767282dd0ddb6
Cr-Commit-Position: refs/heads/master@{#439840}

Powered by Google App Engine
This is Rietveld 408576698