commit | 4e9a0f74389e0745855d939ac9ef900fbaa68fec | [log] [tgz] |
---|---|---|
author | jam <jam@chromium.org> | Fri Dec 04 20:09:09 2015 |
committer | Commit bot <commit-bot@chromium.org> | Fri Dec 04 20:09:58 2015 |
tree | d85dd74b5d7fd06e7abb3f9c97fb5e34da3124ea | |
parent | bf8ab5f6cfc6776a4cf0e59d68ff911d598c7c17 [diff] |
Revert of Fix TSAN error in skia/chromium ref count debug checks (patchset #1 id:1 of https://codereview.chromium.org/1494633002/ ) Reason for revert: Broke https://build.chromium.org/p/chromium.linux/builders/Linux%20Builder%20%28dbg%29/builds/96882/steps/compile/logs/stdio Looks like analyze didn't do any compiles because sk_ref_cnt_ext_debug.h is not in any build files. Please ensure sk_ref_cnt_ext_debug.h is listed in gyp and build.gn files, otherwise analyze won't know to build any targets when that file changes. Original issue's description: > Fix TSAN error in skia/chromium ref count debug checks > > Using std::atomic to make debug checks thread safe. > > BUG=509693 > R=senorblanco@chromium.org > > Committed: https://crrev.com/0d030275667855c06be6cfc2ac10fb434d7ed771 > Cr-Commit-Position: refs/heads/master@{#363274} TBR=senorblanco@chromium.org,junov@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=509693 Review URL: https://codereview.chromium.org/1500033003 Cr-Commit-Position: refs/heads/master@{#363277}