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

Reland of Bug Fix: Data Race For WebMediaPlayerMS' Frame Count Variable (patchset #1 id:1 of https://codereview.chromium.org/2777183009/ )

Reason for revert:
By testing, this CL is not the cause of increase of chrome_public_test_apk run time.

Original issue's description:
> 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

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/2789973002
Cr-Commit-Position: refs/heads/master@{#461555}
Committed: https://chromium.googlesource.com/chromium/src/+/278b26a717794d67f361145e0a4a72a758839d8a

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

The CQ bit was checked by qiangchen@chromium.org

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

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

[commit-bot: I haz the power, 2017-03-31 23:04:46]

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-04-01 00:19:09]

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

[commit-bot: I haz the power, 2017-04-01 00:19:11]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[qiangchen, 2017-04-03 15:54:27]

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

[commit-bot: I haz the power, 2017-04-03 15:54:48]

Dry run: 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-04-03 16:44:52]

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

[commit-bot: I haz the power, 2017-04-03 16:44:53]

Dry run: This issue passed the CQ dry run.

[DaleCurtis, 2017-04-03 19:13:04]

lgtm

[qiangchen, 2017-04-03 22:04:53]

The CQ bit was checked by qiangchen@chromium.org

[commit-bot: I haz the power, 2017-04-03 22:05:47]

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-04-03 22:13:57]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491257093939480, "parent_rev":
"626b2f96095ae42d7805126e2194a373c630299e", "commit_rev":
"278b26a717794d67f361145e0a4a72a758839d8a"}

[commit-bot: I haz the power, 2017-04-03 22:14:51]

Description was changed from

==========
Reland of Bug Fix: Data Race For WebMediaPlayerMS' Frame Count Variable
(patchset #1 id:1 of https://codereview.chromium.org/2777183009/ )

Reason for revert:
By testing, this CL is not the cause of increase of chrome_public_test_apk run
time.

Original issue's description:
> 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...

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

to

==========
Reland of Bug Fix: Data Race For WebMediaPlayerMS' Frame Count Variable
(patchset #1 id:1 of https://codereview.chromium.org/2777183009/ )

Reason for revert:
By testing, this CL is not the cause of increase of chrome_public_test_apk run
time.

Original issue's description:
> 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...

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/2789973002
Cr-Commit-Position: refs/heads/master@{#461555}
Committed:
https://chromium.googlesource.com/chromium/src/+/278b26a717794d67f361145e0a4a...
==========

[commit-bot: I haz the power, 2017-04-03 22:14:52]

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