Generate OnPipelineSeeked event for seek to currenttime in pause state

Generate OnPipelineSeeked event for seek to currenttime in pause state

Pipeline generates OnPipelineSeeked and OnPipelineBufferingStateChanged
events in response to seek operations. There is an event generated already
for OnPipelineBufferingStateChanged but missing one for OnPipelineSeeked.

Follow up fix for https://codereview.chromium.org/740663002/
Required for landing https://codereview.chromium.org/456343002/

Committed: https://crrev.com/8f4a375654e361ff368474f19d0221cb751cee68
Cr-Commit-Position: refs/heads/master@{#308263}

[Srirama, 2014-12-12 14:56:37]

srirama.m@samsung.com changed reviewers:
+ dalecurtis@chromium.org, philipj@opera.com

[Srirama, 2014-12-12 14:56:37]

The test media/controls-restrained-media-controller.html is failing in
https://codereview.chromium.org/456343002/
We need this patch to mimic the actual seek scenario.
I have added OnPipelineSeeked mock event which is missing.
PTAL

[DaleCurtis, 2014-12-12 23:00:52]

https://codereview.chromium.org/791233003/diff/1/media/blink/webmediaplayer_i...
File media/blink/webmediaplayer_impl.cc (right):

https://codereview.chromium.org/791233003/diff/1/media/blink/webmediaplayer_i...
media/blink/webmediaplayer_impl.cc:330: // In paused state ignore the seek
operations to current time and mock
mock has a special connotation in Chrome, so keep this as generate.

https://codereview.chromium.org/791233003/diff/1/media/blink/webmediaplayer_i...
media/blink/webmediaplayer_impl.cc:338: FROM_HERE,
base::Bind(&WebMediaPlayerImpl::OnPipelineSeeked,
This may result in changing the paused_time_ if GetMediaTime() has drifted a bit
since the pause.  No idea if that's possible.

[DaleCurtis, 2014-12-12 23:01:06]

lgtm % comment change.

[Srirama, 2014-12-13 02:23:43]

Patchset #2 (id:20001) has been deleted

[Srirama, 2014-12-13 04:23:42]

https://codereview.chromium.org/791233003/diff/1/media/blink/webmediaplayer_i...
File media/blink/webmediaplayer_impl.cc (right):

https://codereview.chromium.org/791233003/diff/1/media/blink/webmediaplayer_i...
media/blink/webmediaplayer_impl.cc:330: // In paused state ignore the seek
operations to current time and mock
On 2014/12/12 23:00:52, DaleCurtis wrote:
> mock has a special connotation in Chrome, so keep this as generate.

Done.

https://codereview.chromium.org/791233003/diff/1/media/blink/webmediaplayer_i...
media/blink/webmediaplayer_impl.cc:338: FROM_HERE,
base::Bind(&WebMediaPlayerImpl::OnPipelineSeeked,
On 2014/12/12 23:00:52, DaleCurtis wrote:
> This may result in changing the paused_time_ if GetMediaTime() has drifted a
bit
> since the pause.  No idea if that's possible.

ok

[Srirama, 2014-12-13 05:21:40]

The CQ bit was checked by srirama.m@samsung.com

[commit-bot: I haz the power, 2014-12-13 05:22:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/791233003/40001

[commit-bot: I haz the power, 2014-12-13 08:16:24]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2014-12-13 08:17:22]

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