Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2651683002:
Remove layout test suppression added for Skia DEPS roll (Closed)
|
Created:
3 years, 11 months ago by robertphillips Modified:
3 years, 11 months ago CC:
chromium-reviews, blink-reviews, hcm1 Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
DescriptionRemove layout test suppression added for Skia DEPS roll
BUG=683312
Review-Url: https://codereview.chromium.org/2651683002
Cr-Commit-Position: refs/heads/master@{#445416}
Committed: https://chromium.googlesource.com/chromium/src/+/f34f92725f250f7028e43df9a40a3813cc82d99a
Patch Set 1 #
Messages
Total messages: 18 (14 generated)
Description was changed from ========== Remove layout test suppression added for Skia DEPS roll BUG=683312 ========== to ========== Remove layout test suppression added for Skia DEPS roll BUG=683312 ==========
robertphillips@google.com changed reviewers: + fmalita@chromium.org, reed@chromium.org
The CQ bit was checked by robertphillips@google.com 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: win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...) win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...) win_clang on master.tryserver.chromium.win (JOB_FAILED, no build URL)
reed@google.com changed reviewers: + reed@google.com
lgtm
The CQ bit was checked by robertphillips@google.com 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.
The CQ bit was checked by robertphillips@google.com
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": 1, "attempt_start_ts": 1485196239106060, "parent_rev":
"cf7b58737b355ece16d0511aa43c95d6d11d0445", "commit_rev":
"f34f92725f250f7028e43df9a40a3813cc82d99a"}
Message was sent while issue was closed.
Description was changed from ========== Remove layout test suppression added for Skia DEPS roll BUG=683312 ========== to ========== Remove layout test suppression added for Skia DEPS roll BUG=683312 Review-Url: https://codereview.chromium.org/2651683002 Cr-Commit-Position: refs/heads/master@{#445416} Committed: https://chromium.googlesource.com/chromium/src/+/f34f92725f250f7028e43df9a40a... ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/f34f92725f250f7028e43df9a40a... |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
