Suppress TSAN for cc::VideoLayerImpl::WillDraw

Suppress TSAN for cc::VideoLayerImpl::WillDraw

This function grabs locks in different orders depending on the order
of the layers being drawn. But they are all released before being any
are grabbed again, and more than one lock grab only occurs on the
compositor thread.

R=pbos@chromium.org,earthdok
BUG=476529
Committed: https://crrev.com/6e85bb2fcfcee27c0bfb78e1492565e6dfe9f11b
Cr-Commit-Position: refs/heads/master@{#327304}

[danakj, 2015-04-23 22:26:53]

danakj@chromium.org changed reviewers:
+ earthdok@chromium.org

[pbos, 2015-04-24 08:09:54]

The CL description should read something like "acquiring more than one lock only
ever happens on one thread, and is therefore safe".

https://codereview.chromium.org/1081103009/diff/1/build/sanitizers/tsan_suppr...
File build/sanitizers/tsan_suppressions.cc (right):

https://codereview.chromium.org/1081103009/diff/1/build/sanitizers/tsan_suppr...
build/sanitizers/tsan_suppressions.cc:161: "race:cc::VideoLayerImpl::WillDraw\n"
this should read deadlock:cc...

[danakj, 2015-04-24 16:45:14]

PTAL

[danakj, 2015-04-24 20:11:36]

ping?

[pbos, 2015-04-24 21:32:27]

lgtm

On Fri, Apr 24, 2015, 22:11  <danakj@chromium.org> wrote:

> ping?
>
> https://codereview.chromium.org/1081103009/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[danakj, 2015-04-24 21:33:16]

earthdok can you OWNERS?

[pbos, 2015-04-25 09:15:45]

apparently, I'm not allowed to do that by email, lgtm

[danakj, 2015-04-25 17:12:07]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2015-04-25 17:12:24]

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

[commit-bot: I haz the power, 2015-04-25 17:12:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-25 17:12:27]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Alexander Potapenko, 2015-04-28 06:14:40]

On 2015/04/25 17:12:27, I haz the power (commit-bot) wrote:
> No LGTM from a valid reviewer yet. Only full committers are accepted.
> Even if an LGTM may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

lgtm

[danakj, 2015-04-28 16:46:00]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2015-04-28 16:46:05]

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

[commit-bot: I haz the power, 2015-04-28 16:53:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-28 16:54:17]

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