Call CheckTrialGroup only under lock.

Call CheckTrialGroup only under lock.

Since this code can be called from multiple
threads, we should hold a lock when calling
CheckTrialGroup since it modifies a std::map.

The other call site was already holding the
lock.

BUG=595264
Committed: https://crrev.com/cba00314e57ab31bc5a7f766e9bb8d5760d106ac
Cr-Commit-Position: refs/heads/master@{#381535}

[Alexei Svitkine (slow), 2016-03-16 15:11:41]

asvitkine@chromium.org changed reviewers:
+ rkaplow@chromium.org

[rkaplow, 2016-03-16 18:59:25]

lgtm

[Alexei Svitkine (slow), 2016-03-16 19:03:39]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2016-03-16 19:03:57]

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

[commit-bot: I haz the power, 2016-03-16 20:53:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-16 20:56:04]

Description was changed from

==========
Call CheckTrialGroup only under lock.

Since this code can be called from multiple
threads, we should hold a lock when calling
CheckTrialGroup since it modifies a std::map.

The other call site was already holding the
lock.

BUG=595264
==========

to

==========
Call CheckTrialGroup only under lock.

Since this code can be called from multiple
threads, we should hold a lock when calling
CheckTrialGroup since it modifies a std::map.

The other call site was already holding the
lock.

BUG=595264

Committed: https://crrev.com/cba00314e57ab31bc5a7f766e9bb8d5760d106ac
Cr-Commit-Position: refs/heads/master@{#381535}
==========

[commit-bot: I haz the power, 2016-03-16 20:56:05]

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