Invoke the seek callback during abort if needed.

Invoke the seek callback during abort if needed.

BUG=392281
TEST=ChunkDemuxerTest.SeekCompleteDuringAbort

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=281965

[damienv1, 2014-07-08 21:16:20]

Please take a look. Thanks.

[damienv1, 2014-07-08 21:18:39]

Note: I used the exact same logic as what is done in ChunkDemuxer::AppendData.

However, I think we don't need to check the previous seek state, could just be:
if (!seek_cb_.is_null() && !IsSeekWaitingForData_Locked())
  base::ResetAndReturn(&seek_cb_).Run(PIPELINE_OK);

[acolwell GONE FROM CHROMIUM, 2014-07-08 22:06:31]

lgtm

[damienv1, 2014-07-08 23:17:40]

The CQ bit was checked by damienv@chromium.org

[damienv1, 2014-07-08 23:18:11]

The CQ bit was unchecked by damienv@chromium.org

[damienv1, 2014-07-09 01:04:44]

The CQ bit was checked by damienv@chromium.org

[commit-bot: I haz the power, 2014-07-09 01:06:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/damienv@chromium.org/374203003/1

[commit-bot: I haz the power, 2014-07-09 05:52:51]

Change committed as 281965

[wolenetz, 2014-07-12 00:06:32]

https://codereview.chromium.org/374203003/diff/1/media/filters/chunk_demuxer_...
File media/filters/chunk_demuxer_unittest.cc (right):

https://codereview.chromium.org/374203003/diff/1/media/filters/chunk_demuxer_...
media/filters/chunk_demuxer_unittest.cc:2976: 
Hmm. Reviewing this post-commit, but it looks like we never verify the following
here:
1. Seek should not immediately complete
2. Seek should complete after the Abort(), below.

See AppendDataAfterSeek test for one example of how to do this verification.

Best for a separate CL at this point - sorry for belated CR comment on this one.