|
|
DescriptionStop doing at-exit cleanup of lazy pointers.
There are only a fixed number of them, and we're already intentionally leaking in Release.
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug-ASAN-Trybot
BUG=skia:2940
Committed: https://skia.googlesource.com/skia/+/c7eb58c8b46c72cca3cea1582c7f860293a1cbe9
Patch Set 1 #Patch Set 2 : rebase #Patch Set 3 : re-rebase #Messages
Total messages: 23 (9 generated)
mtklein@google.com changed reviewers: + bungeman@google.com, mtklein@google.com, reed@google.com
The CQ bit was checked by mtklein@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/606013004/1
Note for Reviewers: The CQ is waiting for an approval. If you believe that the CL is not ready yet, or if you would like to L-G-T-M with comments then please uncheck the CQ checkbox. Waiting for LGTM from a valid reviewer till 2014-09-27 01:01 UTC
Note for Reviewers: The CQ is waiting for an approval. If you believe that the CL is not ready yet, or if you would like to L-G-T-M with comments then please uncheck the CQ checkbox. Waiting for LGTM from a valid reviewer till 2014-09-27 01:04 UTC
The CQ bit was unchecked by mtklein@google.com
The CQ bit was checked by mtklein@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/606013004/1
Note for Reviewers: The CQ is waiting for an approval. If you believe that the CL is not ready yet, or if you would like to L-G-T-M with comments then please uncheck the CQ checkbox. Waiting for LGTM from a valid reviewer till 2014-09-27 01:34 UTC
The CQ bit was unchecked by commit-bot@chromium.org
No LGTM from a valid reviewer yet. Please ask for an LGTM from a full Skia committer from https://skia.googlesource.com/skia/+/master/CQ_COMMITTERS
reed@google.com changed reviewers: - reed@google.com
deferring to Ben
On 2014/09/29 12:39:41, reed1 wrote: > deferring to Ben Ping!
The Sheriff-hat-wearing aspect of me is going to land this. Let's follow up if you think there's a problem.
The CQ bit was checked by mtklein@google.com
lgtm
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/606013004/40001
The CQ bit was unchecked by mtklein@google.com
The CQ bit was checked by mtklein@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/606013004/40001
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as c7eb58c8b46c72cca3cea1582c7f860293a1cbe9 |