Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2531563002:
[wrapper-tracing] Fix leak in RemotePlayback round 2 (Closed)
|
Created:
4 years ago by Michael Lippautz Modified:
4 years ago CC:
blink-reviews, chromium-reviews Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
Description[wrapper-tracing] Fix leak in RemotePlayback round 2
BUG=chromium:667746
Committed: https://crrev.com/68e0614944ce7c45de33bc8722df414eddf7a0a8
Cr-Commit-Position: refs/heads/master@{#434382}
Patch Set 1 #
Messages
Total messages: 26 (16 generated)
Description was changed from ========== [wrapper-tracing] Fix leak in RemotePlayback round 2 BUG= ========== to ========== [wrapper-tracing] Fix leak in RemotePlayback round 2 BUG=chromium:667746 ==========
mlippautz@chromium.org changed reviewers: + haraken@chromium.org, hlopko@google.com
Fixes the last test that triggers the leak detector
The CQ bit was checked by mlippautz@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...
hlopko@chromium.org changed reviewers: + hlopko@chromium.org
lgtm
LGTM
The CQ bit was unchecked by mlippautz@chromium.org
The CQ bit was checked by mlippautz@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 commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by mlippautz@chromium.org
The CQ bit was unchecked by mlippautz@chromium.org
The CQ bit was checked by mlippautz@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 commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by mlippautz@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": 1480013061515850, "parent_rev":
"f5bb6769108eb79a8a35e1b8e6cb5f71ed9baf65", "commit_rev":
"ef43545d79a9ea2c39f82b7154fa188f7ac574d3"}
Message was sent while issue was closed.
Description was changed from ========== [wrapper-tracing] Fix leak in RemotePlayback round 2 BUG=chromium:667746 ========== to ========== [wrapper-tracing] Fix leak in RemotePlayback round 2 BUG=chromium:667746 ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== [wrapper-tracing] Fix leak in RemotePlayback round 2 BUG=chromium:667746 ========== to ========== [wrapper-tracing] Fix leak in RemotePlayback round 2 BUG=chromium:667746 Committed: https://crrev.com/68e0614944ce7c45de33bc8722df414eddf7a0a8 Cr-Commit-Position: refs/heads/master@{#434382} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/68e0614944ce7c45de33bc8722df414eddf7a0a8 Cr-Commit-Position: refs/heads/master@{#434382} |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
