Fix DCHECK when track status changes in non-playing state

Fix DCHECK when track status changes in non-playing state

Reland of https://codereview.chromium.org/2269313002
That CL added a new unit test TrackStatusChangesWhileSuspended that
turned out to be flaky. I have investigated the flakiness and tracked
it down to a race condition in Renderer destruction when pipeline is
being suspended. That race condition has been recently fixed (see
https://codereview.chromium.org/2355173002 and crbug.com/638018) and
the new test is not flaky anymore. I've confirmed that the test becomes
flaky again when I revert commit bdf51ba921ccbc locally.

BUG=640362
Committed: https://crrev.com/09ed890b572504e099b9e974c29c69d168881a6a
Cr-Commit-Position: refs/heads/master@{#421669}

[servolk, 2016-09-28 21:26:29]

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

[commit-bot: I haz the power, 2016-09-28 21:26:59]

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

[servolk, 2016-09-28 21:28:36]

Description was changed from

==========
Fix DCHECK when track status changes in non-playing state

Reland of https://codereview.chromium.org/2269313002
That CL added a new unit test TrackStatusChangesWhileSuspended that
turned out to be flaky. I have investigated the flakiness and tracked
it down to a race condition in Renderer destruction when pipeline is
being suspended. That race condition has been recently fixed (see
https://codereview.chromium.org/2355173002 and crbug.com/638018) and
the new test is not flaky anymore. I've confirmed that the test becomes
flaky again when I revert commit bdf51ba921ccbc locally.

BUG=640362
==========

to

==========
Fix DCHECK when track status changes in non-playing state

Reland of https://codereview.chromium.org/2269313002
That CL added a new unit test TrackStatusChangesWhileSuspended that
turned out to be flaky. I have investigated the flakiness and tracked
it down to a race condition in Renderer destruction when pipeline is
being suspended. That race condition has been recently fixed (see
https://codereview.chromium.org/2355173002 and crbug.com/638018) and
the new test is not flaky anymore. I've confirmed that the test becomes
flaky again when I revert commit bdf51ba921ccbc locally.

BUG=640362
==========

[servolk, 2016-09-28 21:28:37]

servolk@chromium.org changed reviewers:
+ chcunningham@chromium.org, dalecurtis@chromium.org

[DaleCurtis, 2016-09-28 21:34:49]

lgtm

[servolk, 2016-09-28 22:39:51]

The CQ bit was unchecked by servolk@chromium.org

[servolk, 2016-09-28 22:39:52]

The CQ bit was checked by servolk@chromium.org

[commit-bot: I haz the power, 2016-09-28 22:40:04]

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

[chcunningham, 2016-09-28 22:44:54]

lgtm

[commit-bot: I haz the power, 2016-09-28 22:46:46]

Description was changed from

==========
Fix DCHECK when track status changes in non-playing state

Reland of https://codereview.chromium.org/2269313002
That CL added a new unit test TrackStatusChangesWhileSuspended that
turned out to be flaky. I have investigated the flakiness and tracked
it down to a race condition in Renderer destruction when pipeline is
being suspended. That race condition has been recently fixed (see
https://codereview.chromium.org/2355173002 and crbug.com/638018) and
the new test is not flaky anymore. I've confirmed that the test becomes
flaky again when I revert commit bdf51ba921ccbc locally.

BUG=640362
==========

to

==========
Fix DCHECK when track status changes in non-playing state

Reland of https://codereview.chromium.org/2269313002
That CL added a new unit test TrackStatusChangesWhileSuspended that
turned out to be flaky. I have investigated the flakiness and tracked
it down to a race condition in Renderer destruction when pipeline is
being suspended. That race condition has been recently fixed (see
https://codereview.chromium.org/2355173002 and crbug.com/638018) and
the new test is not flaky anymore. I've confirmed that the test becomes
flaky again when I revert commit bdf51ba921ccbc locally.

BUG=640362
==========

[commit-bot: I haz the power, 2016-09-28 22:46:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-28 22:48:25]

Description was changed from

==========
Fix DCHECK when track status changes in non-playing state

Reland of https://codereview.chromium.org/2269313002
That CL added a new unit test TrackStatusChangesWhileSuspended that
turned out to be flaky. I have investigated the flakiness and tracked
it down to a race condition in Renderer destruction when pipeline is
being suspended. That race condition has been recently fixed (see
https://codereview.chromium.org/2355173002 and crbug.com/638018) and
the new test is not flaky anymore. I've confirmed that the test becomes
flaky again when I revert commit bdf51ba921ccbc locally.

BUG=640362
==========

to

==========
Fix DCHECK when track status changes in non-playing state

Reland of https://codereview.chromium.org/2269313002
That CL added a new unit test TrackStatusChangesWhileSuspended that
turned out to be flaky. I have investigated the flakiness and tracked
it down to a race condition in Renderer destruction when pipeline is
being suspended. That race condition has been recently fixed (see
https://codereview.chromium.org/2355173002 and crbug.com/638018) and
the new test is not flaky anymore. I've confirmed that the test becomes
flaky again when I revert commit bdf51ba921ccbc locally.

BUG=640362

Committed: https://crrev.com/09ed890b572504e099b9e974c29c69d168881a6a
Cr-Commit-Position: refs/heads/master@{#421669}
==========

[commit-bot: I haz the power, 2016-09-28 22:48:26]

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