Fix TSAN error in skia/chromium ref count debug checks

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}

Committed: https://crrev.com/3e471195b0826a8b8228fefd378930f65c2671b0
Cr-Commit-Position: refs/heads/master@{#366750}

[Justin Novosad, 2015-12-04 15:44:20]

PTAL

[Stephen White, 2015-12-04 19:02:27]

LGTM

[Justin Novosad, 2015-12-04 19:36:17]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2015-12-04 19:37:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1494633002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1494633002/1

[commit-bot: I haz the power, 2015-12-04 19:52:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-04 19:53:13]

Description was changed from

==========
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
==========

to

==========
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}
==========

[commit-bot: I haz the power, 2015-12-04 19:53:14]

Patchset 1 (id:??) landed as
https://crrev.com/0d030275667855c06be6cfc2ac10fb434d7ed771
Cr-Commit-Position: refs/heads/master@{#363274}

[jam, 2015-12-04 20:05:26]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1500033003/ by jam@chromium.org.

The reason for reverting is: Broke
https://build.chromium.org/p/chromium.linux/builders/Linux%20Builder%20%28dbg...

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..

[Justin Novosad, 2015-12-23 15:23:03]

Description was changed from

==========
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}
==========

to

==========
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}
==========

[Justin Novosad, 2015-12-23 15:23:07]

The CQ bit was checked by junov@chromium.org

[Justin Novosad, 2015-12-23 15:23:08]

The patchset sent to the CQ was uploaded after l-g-t-m from
senorblanco@chromium.org
Link to the patchset: https://codereview.chromium.org/1494633002/#ps20001
(title: "fix build deps")

[commit-bot: I haz the power, 2015-12-23 15:23:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1494633002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1494633002/20001

[commit-bot: I haz the power, 2015-12-23 16:21:39]

Description was changed from

==========
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}
==========

to

==========
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}
==========

[commit-bot: I haz the power, 2015-12-23 16:21:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-23 16:22:51]

Description was changed from

==========
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}
==========

to

==========
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}

Committed: https://crrev.com/3e471195b0826a8b8228fefd378930f65c2671b0
Cr-Commit-Position: refs/heads/master@{#366750}
==========

[commit-bot: I haz the power, 2015-12-23 16:22:51]

Patchset 2 (id:??) landed as
https://crrev.com/3e471195b0826a8b8228fefd378930f65c2671b0
Cr-Commit-Position: refs/heads/master@{#366750}