Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2827643004:
[LayoutNG] Add FlagExpectations for LayoutNG 1/3 (Closed)
Description[LayoutNG] Add FlagExpectations for LayoutNG 1/3
This patch adds the FlagExpectations file for LayoutNG.
The patch is split into 3 CLs because it hits the max limit of CQ.
There are many flaky tests that may need multiple passes until it is
stabilized.
BUG=591099
Review-Url: https://codereview.chromium.org/2827643004
Cr-Commit-Position: refs/heads/master@{#466571}
Committed: https://chromium.googlesource.com/chromium/src/+/5d44f961b168a6cff0f822455b03cbca3eb44855
Patch Set 1 #Patch Set 2 : Add more results #Patch Set 3 : Add more results #Patch Set 4 : Add more results #Patch Set 5 : Add more results #Patch Set 6 : Add more results #Patch Set 7 : Add more results #Patch Set 8 : Add more results #Patch Set 9 : Add more results #Patch Set 10 : Add more results #Patch Set 11 : Reduced to 9500 lines #Patch Set 12 : Removed recently removed tests #Messages
Total messages: 29 (25 generated)
The CQ bit was checked by kojii@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Your CL can not be processed by CQ because of: * Failed to parse additional trybots
Description was changed from ========== ngbot BUG= CQ_INCLUDE_TRYBOTS=linux_layout_tests_layout_ng ========== to ========== ngbot BUG= CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng ==========
The CQ bit was checked by kojii@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_layout_tests_layout_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_layout_te...)
The CQ bit was checked by kojii@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by kojii@chromium.org
The CQ bit was checked by kojii@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
Description was changed from ========== ngbot BUG= CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng ========== to ========== [LayoutNG] Add FlagExpectations for LayoutNG 1/3 This patch adds the FlagExpectations file for LayoutNG. The patch is split into 3 CLs because it hits the max limit of CQ. There are many flaky tests that may need multiple passes until it is stabilized. BUG=591099 ==========
kojii@chromium.org changed reviewers: + eae@chromium.org
PTAL. After "exists early" was handled, many tests are flaky that we might need several passes until we collect all rare failures.
The CQ bit was checked by kojii@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
LGTM, thank you for doing this!
The CQ bit was checked by eae@chromium.org
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch.
Bot data: {"patchset_id": 180002, "attempt_start_ts": 1492991359683810,
"parent_rev": "beb19669855bab47cda70ea40fbbe154eb3bfdc5", "commit_rev":
"5d44f961b168a6cff0f822455b03cbca3eb44855"}
Message was sent while issue was closed.
Description was changed from ========== [LayoutNG] Add FlagExpectations for LayoutNG 1/3 This patch adds the FlagExpectations file for LayoutNG. The patch is split into 3 CLs because it hits the max limit of CQ. There are many flaky tests that may need multiple passes until it is stabilized. BUG=591099 ========== to ========== [LayoutNG] Add FlagExpectations for LayoutNG 1/3 This patch adds the FlagExpectations file for LayoutNG. The patch is split into 3 CLs because it hits the max limit of CQ. There are many flaky tests that may need multiple passes until it is stabilized. BUG=591099 Review-Url: https://codereview.chromium.org/2827643004 Cr-Commit-Position: refs/heads/master@{#466571} Committed: https://chromium.googlesource.com/chromium/src/+/5d44f961b168a6cff0f822455b03... ==========
Message was sent while issue was closed.
Committed patchset #12 (id:180002) as https://chromium.googlesource.com/chromium/src/+/5d44f961b168a6cff0f822455b03... |
