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

Issue 2803633005: Move Paint Team TestExpectations lines into appropriate sections (Closed)

Created:
3 years, 8 months ago by Stephen Chennney
Modified:
3 years, 8 months ago
Reviewers:
chrishtr
CC:
chromium-reviews, blink-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Move Paint Team TestExpectations lines into appropriate sections This completes a review of all the Paint Team owned layout test failures. With this patch tests are grouped so we know which ones might be worth investigating in the future and so we can more easily incorporate new failures. There is one outstanding patch to be comnitted rebasing one set of tests. TBR=chrishtr@chromium.org BUG=309675, 504613, 517449, 549742, 602110, 606302, 614910, 676561 Review-Url: https://codereview.chromium.org/2803633005 Cr-Commit-Position: refs/heads/master@{#462231} Committed: https://chromium.googlesource.com/chromium/src/+/858bfb659e50d2d2debebd92847c59b884848ae4

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -23 lines) Patch
M third_party/WebKit/LayoutTests/NeverFixTests View 1 chunk +7 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/TestExpectations View 2 chunks +18 lines, -23 lines 0 comments Download

Messages

Total messages: 13 (9 generated)
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/2803633005/1
3 years, 8 months ago (2017-04-05 20:41:58 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_compile_dbg_ng/builds/389419)
3 years, 8 months ago (2017-04-05 20:47:57 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/2803633005/1
3 years, 8 months ago (2017-04-05 21:05:34 UTC) #10
commit-bot: I haz the power
3 years, 8 months ago (2017-04-05 21:53:39 UTC) #13
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/858bfb659e50d2d2debebd92847c...

Powered by Google App Engine
This is Rietveld 408576698