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

Issue 2390323002: Remove non-flaky lines in TestExpectations (Closed)

Created:
4 years, 2 months ago by bokan
Modified:
4 years, 2 months ago
Reviewers:
qyearsley
CC:
chromium-reviews, blink-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove non-flaky lines in TestExpectations This CL was created using the update-test-expectations script. Contact bokan@ if you believe a line was removed in error. BUG=521096, 445100, 518929, 248938, 649063, 570894 Committed: https://crrev.com/05cbd5d7a01746aad60e6e2350d1d09ce09aa15c Cr-Commit-Position: refs/heads/master@{#423373}

Patch Set 1 #

Patch Set 2 : Rerun due to conflicting changes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -10 lines) Patch
M third_party/WebKit/LayoutTests/TestExpectations View 1 6 chunks +0 lines, -10 lines 0 comments Download

Messages

Total messages: 17 (9 generated)
bokan
Updated run. FYI: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=imported/wpt/webstorage/event_case_sensitive.html,fast/block/float/marquee-shrink-to-avoid-floats.html,http/tests/appcache/remove-cache.html,virtual/threaded/transitions/transition-end-event-prefixed-01.html,virtual/gpu/fast/canvas/canvas-ellipse-circumference.html,virtual/gpu/fast/canvas/canvas-ellipse-circumference-fill.html,virtual/gpu/fast/canvas/canvas-arc-circumference.html,virtual/gpu/fast/canvas/canvas-arc-circumference-fill.html,http/tests/workers/terminate-during-sync-operation-file.html,http/tests/inspector/resource-tree/resource-tree-non-unique-url.html http/tests/appcache/remove-cache.html still fails on an FYI bot but I assume TestExpectations ...
4 years, 2 months ago (2016-10-04 16:39:48 UTC) #2
bokan
ping
4 years, 2 months ago (2016-10-05 20:49:41 UTC) #3
qyearsley
On 2016/10/05 at 20:49:41, bokan wrote: > ping Excellent, LGTM
4 years, 2 months ago (2016-10-05 20:56:01 UTC) #4
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/2390323002/1
4 years, 2 months ago (2016-10-05 20:57:11 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm64_dbg_recipe/builds/140516) linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED, ...
4 years, 2 months ago (2016-10-05 21:01:05 UTC) #8
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/2390323002/20001
4 years, 2 months ago (2016-10-05 22:58:47 UTC) #13
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 2 months ago (2016-10-06 01:13:04 UTC) #15
commit-bot: I haz the power
4 years, 2 months ago (2016-10-06 01:15:19 UTC) #17
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/05cbd5d7a01746aad60e6e2350d1d09ce09aa15c
Cr-Commit-Position: refs/heads/master@{#423373}

Powered by Google App Engine
This is Rietveld 408576698