Revert of Bug Fix: Data Race For WebMediaPlayerMS' Frame Count Variable

Revert of Bug Fix: Data Race For WebMediaPlayerMS' Frame Count Variable (patchset #3 id:60001 of https://codereview.chromium.org/2751353003/ )

Reason for revert:
Reverting the cl to see whether lock contention has increased chrome public test apk time. Thanks qiangchen for understanding!

Original issue's description:
> Bug Fix: Data Race For WebMediaPlayerMS' Frame Count Variable
> 
> On thread sanitizer, we found a data race issue for WebMediaPlayerMS' frame
> count variables |total_frame_count_| and |dropped_frame_count_|. In this CL
> We make those two variables atomic.
> 
> BUG=701712
> 
> Review-Url: https://codereview.chromium.org/2751353003
> Cr-Commit-Position: refs/heads/master@{#457786}
> Committed: https://chromium.googlesource.com/chromium/src/+/4cb6dcab2e3d9b4f14ff3bd5dc8f3fd3a57e19e5

TBR=dalecurtis@chromium.org,qiangchen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=701712

[BigBossZhiling, 2017-03-29 21:02:49]

The CQ bit was checked by hzl@google.com

[BigBossZhiling, 2017-03-29 21:02:50]

Created Revert of Bug Fix: Data Race For WebMediaPlayerMS' Frame Count Variable

[commit-bot: I haz the power, 2017-03-29 21:03:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-29 21:03:45]

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

[commit-bot: I haz the power, 2017-03-29 21:03:46]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[qiangchen, 2017-03-29 21:10:12]

On 2017/03/29 21:03:46, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet.
> CQ run can only be started once the patch has received an L-G-T-M from a full
> committer.
> Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
> full super star committer.
> Committers are members of the group "project-chromium-committers".
> Note that this has nothing to do with OWNERS files.

I've created a revert CL, and it is in CQ now, so you can close this one.

[chromium-reviews, 2017-03-29 21:13:09]

Nice!

On Wed, Mar 29, 2017 at 2:10 PM, <qiangchen@chromium.org> wrote:

> On 2017/03/29 21:03:46, commit-bot: I haz the power wrote:
> > No L-G-T-M from a valid reviewer yet.
> > CQ run can only be started once the patch has received an L-G-T-M from a
> full
> > committer.
> > Even if an L-G-T-M may have been provided, it was from a
> non-committer,_not_ a
> > full super star committer.
> > Committers are members of the group "project-chromium-committers".
> > Note that this has nothing to do with OWNERS files.
>
> I've created a revert CL, and it is in CQ now, so you can close this one.
>
> https://codereview.chromium.org/2783013002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.