Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2912553002:
Remove Leak expectations for the tests not leaking recently on the bots (Closed)
DescriptionRemove Leak expectations for the tests not leaking recently on the bots
... based on the flakiness dashboard.
BUG=364398, 301515, 327574, 629058, 364417, 506529, 506754, 594309
Review-Url: https://codereview.chromium.org/2912553002
Cr-Commit-Position: refs/heads/master@{#475477}
Committed: https://chromium.googlesource.com/chromium/src/+/5a1339aa58b7cb26f064fcb0f6d5d600390b50e8
Patch Set 1 #
Dependent Patchsets: Messages
Total messages: 29 (21 generated)
The CQ bit was checked by hiroshige@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...
Description was changed from ========== Remove Leak expectations for the tests not leaking recently on the bots BUG= ========== to ========== Remove Leak expectations for the tests not leaking recently on the bots ... based on the flakiness dashboard. BUG=None ==========
PTAL > Today's sheriffs.
this seems like a risky change to land on Friday afternoon, do you wait until next week to land?
On 2017/05/26 21:30:07, nednguyen wrote: > this seems like a risky change to land on Friday afternoon, do you wait until > next week to land? Sure, sounds reasonable.
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 hiroshige@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.
The CQ bit was checked by hiroshige@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.
kouhei@chromium.org changed reviewers: + kouhei@chromium.org
lgtm
The CQ bit was checked by hiroshige@chromium.org
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 hiroshige@chromium.org
lgtm
Description was changed from ========== Remove Leak expectations for the tests not leaking recently on the bots ... based on the flakiness dashboard. BUG=None ========== to ========== Remove Leak expectations for the tests not leaking recently on the bots ... based on the flakiness dashboard. BUG=364398, 301515, 327574, 629058, 364417, 506529, 506754, 594309 ==========
The CQ bit was checked by hiroshige@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": 1, "attempt_start_ts": 1496135900091320, "parent_rev":
"2c836f9a6b7c784ca5c4454be1f167071b390eca", "commit_rev":
"5a1339aa58b7cb26f064fcb0f6d5d600390b50e8"}
Message was sent while issue was closed.
Description was changed from ========== Remove Leak expectations for the tests not leaking recently on the bots ... based on the flakiness dashboard. BUG=364398, 301515, 327574, 629058, 364417, 506529, 506754, 594309 ========== to ========== Remove Leak expectations for the tests not leaking recently on the bots ... based on the flakiness dashboard. BUG=364398, 301515, 327574, 629058, 364417, 506529, 506754, 594309 Review-Url: https://codereview.chromium.org/2912553002 Cr-Commit-Position: refs/heads/master@{#475477} Committed: https://chromium.googlesource.com/chromium/src/+/5a1339aa58b7cb26f064fcb0f6d5... ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/5a1339aa58b7cb26f064fcb0f6d5... |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
