Use atomics when accessing m_didSetLooping

Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically. The audio thread was reading this while the
main thread could be writing it.

Manually verified the test case from the bug that this fix works. (Need to reload the page a few times to trigger the original bug.)

BUG=676874
TEST=

Committed: https://crrev.com/310673a6eca24e30371c29593b4385fd3983ad1d
Cr-Commit-Position: refs/heads/master@{#441304}

[Raymond Toy, 2017-01-03 20:49:45]

Description was changed from

==========
Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically.

BUG=676874
TEST=
==========

to

==========
Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically. The audio thread was reading this while the 
main thread could be writing it.

BUG=676874
TEST=
==========

[Raymond Toy, 2017-01-03 20:50:06]

rtoy@chromium.org changed reviewers:
+ hongchan@chromium.org

[Raymond Toy, 2017-01-03 20:50:07]

PTAL

[Raymond Toy, 2017-01-03 20:55:12]

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

[commit-bot: I haz the power, 2017-01-03 20:55:40]

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

[hongchan, 2017-01-03 21:22:14]

lgtm

[Raymond Toy, 2017-01-03 21:31:08]

Description was changed from

==========
Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically. The audio thread was reading this while the 
main thread could be writing it.

BUG=676874
TEST=
==========

to

==========
Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically. The audio thread was reading this while the 
main thread could be writing it.

Manually verified the test case from the bug that this fix works. (Need to
reload the page a few times to trigger the original bug.)

BUG=676874
TEST=
==========

[commit-bot: I haz the power, 2017-01-03 22:59:55]

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

[commit-bot: I haz the power, 2017-01-03 22:59:56]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Raymond Toy, 2017-01-03 23:45:11]

The CQ bit was checked by rtoy@chromium.org

[Raymond Toy, 2017-01-03 23:45:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
hongchan@chromium.org
Link to the patchset: https://codereview.chromium.org/2615463002/#ps20001
(title: "Fix typo.")

[commit-bot: I haz the power, 2017-01-03 23:45:48]

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-01-04 02:56:25]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483487111867990,
"parent_rev": "24045840a7f8dbda2280486c952dff2918fbd54f", "commit_rev":
"7c6897f7d90084df31431811bee55eeb0070ef4f"}

[commit-bot: I haz the power, 2017-01-04 02:56:53]

Description was changed from

==========
Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically. The audio thread was reading this while the 
main thread could be writing it.

Manually verified the test case from the bug that this fix works. (Need to
reload the page a few times to trigger the original bug.)

BUG=676874
TEST=
==========

to

==========
Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically. The audio thread was reading this while the 
main thread could be writing it.

Manually verified the test case from the bug that this fix works. (Need to
reload the page a few times to trigger the original bug.)

BUG=676874
TEST=

Review-Url: https://codereview.chromium.org/2615463002
==========

[commit-bot: I haz the power, 2017-01-04 02:56:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-04 03:00:17]

Description was changed from

==========
Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically. The audio thread was reading this while the 
main thread could be writing it.

Manually verified the test case from the bug that this fix works. (Need to
reload the page a few times to trigger the original bug.)

BUG=676874
TEST=

Review-Url: https://codereview.chromium.org/2615463002
==========

to

==========
Use atomics when accessing m_didSetLooping

Add setter and getter methods for accessing m_didSetLooping so that we
can access it atomically. The audio thread was reading this while the
main thread could be writing it.

Manually verified the test case from the bug that this fix works. (Need to
reload the page a few times to trigger the original bug.)

BUG=676874
TEST=

Committed: https://crrev.com/310673a6eca24e30371c29593b4385fd3983ad1d
Cr-Commit-Position: refs/heads/master@{#441304}
==========

[commit-bot: I haz the power, 2017-01-04 03:00:18]

Patchset 2 (id:??) landed as
https://crrev.com/310673a6eca24e30371c29593b4385fd3983ad1d
Cr-Commit-Position: refs/heads/master@{#441304}