Fixed an issue where tracing events got incorrectly enabled.

Fixed an issue where tracing events got incorrectly enabled.

This specifically happened when the tracing config had excluded specific categories, and a multi-category trace event occured with one of its categories present in the excluded list. The code incorrectly assumed this meant the non-excluded categories were enabled, not accounting for disabled-by-default categories.

R=simonhatch@chromium.org

BUG=560485
Committed: https://crrev.com/a8a6d97c857fd1ce2b94bb710e9af7a24700907a
Cr-Commit-Position: refs/heads/master@{#361667}

[oystein (OOO til 10th of July), 2015-11-24 22:33:35]

Description was changed from

==========
Fixed an issue where tracing events got incorrectly enabled.

This specifically happened when the tracing config had excluded specific
categories, and a multi-category trace event occured with one of its categories
present in the excluded list. The code incorrectly assumed this meant the
non-excluded categories were enabled, not accounting for disabled-by-default
categories.

R=simonhatch@chromium.org

BUG=560485
==========

to

==========
Fixed an issue where tracing events got incorrectly enabled.

This specifically happened when the tracing config had excluded specific
categories, and a multi-category trace event occured with one of its categories
present in the excluded list. The code incorrectly assumed this meant the
non-excluded categories were enabled, not accounting for disabled-by-default
categories.

R=simonhatch@chromium.org

BUG=560485
==========

[oystein (OOO til 10th of July), 2015-11-24 22:33:42]

ptal!

[oystein (OOO til 10th of July), 2015-11-24 22:33:47]

The CQ bit was checked by oysteine@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-11-24 22:37:06]

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

[commit-bot: I haz the power, 2015-11-25 01:55:33]

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

[commit-bot: I haz the power, 2015-11-25 01:55:34]

Dry run: This issue passed the CQ dry run.

[sh, 2015-11-25 15:44:08]

simonhatch@google.com changed reviewers:
+ simonhatch@google.com

[sh, 2015-11-25 15:44:08]

The CQ bit was checked by simonhatch@google.com

[sh, 2015-11-25 15:44:09]

lgtm

[commit-bot: I haz the power, 2015-11-25 15:44:56]

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

[commit-bot: I haz the power, 2015-11-25 15:50:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-25 15:51:22]

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