Ignore sink start attempts when no frames are present.

Ignore sink start attempts when no frames are present.

Buffering state changes may be in flight at the time of the sink
start attempt by RendererImpl. Some of these may have removed all
the frames in the queue.  Ignore sink start in this case.

BUG=601066, 606733
TEST=new unittest

Committed: https://crrev.com/c7aeeb2d491234bfaf07e1ef4e314da09a1f57d3
Cr-Commit-Position: refs/heads/master@{#389933}

[DaleCurtis, 2016-04-26 18:02:42]

dalecurtis@chromium.org changed reviewers:
+ xhwang@chromium.org

[DaleCurtis, 2016-04-26 18:02:42]

Will land a test in a followup once I figure out how, but this will fix the
crash for now.

[DaleCurtis, 2016-04-26 18:42:02]

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

[DaleCurtis, 2016-04-26 18:42:17]

Description was changed from

==========
Ignore sink start attempts when no frames are present.

Buffering state changes may be in flight at the time of the sink
start attempt by RendererImpl. Some of these may have removed all
the frames in the queue.  Ignore sink start in this case.

BUG=606733
TEST=tbd
==========

to

==========
Ignore sink start attempts when no frames are present.

Buffering state changes may be in flight at the time of the sink
start attempt by RendererImpl. Some of these may have removed all
the frames in the queue.  Ignore sink start in this case.

BUG=601066, 606733
TEST=tbd
==========

[commit-bot: I haz the power, 2016-04-26 18:42:56]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1924513003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1924513003/1

[commit-bot: I haz the power, 2016-04-26 19:13:00]

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

[commit-bot: I haz the power, 2016-04-26 19:13:01]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[DaleCurtis, 2016-04-26 19:33:48]

Description was changed from

==========
Ignore sink start attempts when no frames are present.

Buffering state changes may be in flight at the time of the sink
start attempt by RendererImpl. Some of these may have removed all
the frames in the queue.  Ignore sink start in this case.

BUG=601066, 606733
TEST=tbd
==========

to

==========
Ignore sink start attempts when no frames are present.

Buffering state changes may be in flight at the time of the sink
start attempt by RendererImpl. Some of these may have removed all
the frames in the queue.  Ignore sink start in this case.

BUG=601066, 606733
TEST=new unittest
==========

[DaleCurtis, 2016-04-26 19:33:54]

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

[commit-bot: I haz the power, 2016-04-26 19:34:23]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1924513003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1924513003/20001

[xhwang, 2016-04-26 21:20:03]

lgtm

[DaleCurtis, 2016-04-26 21:20:32]

The CQ bit was unchecked by dalecurtis@chromium.org

[DaleCurtis, 2016-04-26 21:20:36]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2016-04-26 21:20:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1924513003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1924513003/20001

[commit-bot: I haz the power, 2016-04-26 22:52:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-26 22:54:17]

Description was changed from

==========
Ignore sink start attempts when no frames are present.

Buffering state changes may be in flight at the time of the sink
start attempt by RendererImpl. Some of these may have removed all
the frames in the queue.  Ignore sink start in this case.

BUG=601066, 606733
TEST=new unittest
==========

to

==========
Ignore sink start attempts when no frames are present.

Buffering state changes may be in flight at the time of the sink
start attempt by RendererImpl. Some of these may have removed all
the frames in the queue.  Ignore sink start in this case.

BUG=601066, 606733
TEST=new unittest

Committed: https://crrev.com/c7aeeb2d491234bfaf07e1ef4e314da09a1f57d3
Cr-Commit-Position: refs/heads/master@{#389933}
==========

[commit-bot: I haz the power, 2016-04-26 22:54:18]

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