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:
Lock contention, increase chrome_public_test_apk run time a lot.

We might need a finer lock.

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
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=701712
Review-Url: https://codereview.chromium.org/2777183009
Cr-Commit-Position: refs/heads/master@{#460556}
Committed: https://chromium.googlesource.com/chromium/src/+/5591e262e49dfbf3f23ad6e54717e9dcd2771fe3

[qiangchen, 2017-03-29 21:02:32]

The CQ bit was checked by qiangchen@chromium.org

[qiangchen, 2017-03-29 21:02:32]

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

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

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 22:31:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490821352554400, "parent_rev":
"defb02ed569af74c118368ac2c8a35b76c71b1b9", "commit_rev":
"5591e262e49dfbf3f23ad6e54717e9dcd2771fe3"}

[commit-bot: I haz the power, 2017-03-29 22:34:33]

Description was changed from

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

Reason for revert:
Lock contention, increase chrome_public_test_apk run time a lot.

We might need a finer lock.

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/+/4cb6dcab2e3d9b4f14ff3bd5dc8f...

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

to

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

Reason for revert:
Lock contention, increase chrome_public_test_apk run time a lot.

We might need a finer lock.

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/+/4cb6dcab2e3d9b4f14ff3bd5dc8f...

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

Review-Url: https://codereview.chromium.org/2777183009
Cr-Commit-Position: refs/heads/master@{#460556}
Committed:
https://chromium.googlesource.com/chromium/src/+/5591e262e49dfbf3f23ad6e54717...
==========

[commit-bot: I haz the power, 2017-03-29 22:34:34]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5591e262e49dfbf3f23ad6e54717...

[qiangchen, 2017-03-31 23:04:12]

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

The reason for reverting is: By testing, this CL is not the cause of increase of
chrome_public_test_apk run time..