|
|
DescriptionSkOncePtr
New abstraction to replace all of: SkOnce, SkLazyPtr, SkLazyFnPtr.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/714a710c428c3166b729b7756199927ab85eb69e
Patch Set 1 #Patch Set 2 : no inheritance #Patch Set 3 : typo #Patch Set 4 : Non-copyable #Patch Set 5 : drop destructor #Messages
Total messages: 25 (11 generated)
mtklein@chromium.org changed reviewers: + herb@google.com
The CQ bit was checked by mtklein@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1301323006/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1301323006/1
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 mtklein@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1301323006/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1301323006/20001
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-Trybot on client.skia (JOB_FAILED, http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)
The CQ bit was checked by mtklein@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1301323006/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1301323006/40001
The CQ bit was checked by mtklein@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1301323006/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1301323006/60001
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 mtklein@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1301323006/80001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1301323006/80001
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
lgtm
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/1301323006/80001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1301323006/80001
Working on the big follow up now.
Message was sent while issue was closed.
Committed patchset #5 (id:80001) as https://skia.googlesource.com/skia/+/714a710c428c3166b729b7756199927ab85eb69e |