Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ConcatMediaSource is failing to restore when ClippingMediaSource is used. #5351

Closed
sravan1213 opened this issue Jan 8, 2019 · 3 comments
Closed
Assignees
Labels

Comments

@sravan1213
Copy link
Contributor

Reproduction steps

1 . Create a ConcatMediaSource with ClippingMediaSource.
2. Play content using above mediaSource.
3. Do player.stop(false)
4. Restore above player instance using player.prepare(mediaSource, false, false).
5. Observation media doesn't restore

Probable fix - listenerNotificationScheduled = false; in releaseSourceInternal of ConcatenatingMediaSource

@vinodkputta
Copy link

@ojw28 Could you please check this issue?

@tonihei tonihei self-assigned this Jan 10, 2019
@tonihei tonihei added the bug label Jan 14, 2019
@tonihei
Copy link
Collaborator

tonihei commented Jan 14, 2019

Thanks for reporting. We'll provide a fix soon.

ojw28 pushed a commit that referenced this issue Jan 14, 2019
We currently forget whether a source is seekable at re-preparation. This was
implemented intentionally this way under the assumption that we really can't seek
until we have loaded the seek map again. However, seek operations are only
allowed after a media period is prepared. So there is no harm in remembering
whether a source is seekable.

This problem currently prevents reusing ClippingMediaSources with
ExtractorMediaSource and a non-zero start clip position.

Issue: #5351
PiperOrigin-RevId: 229169441
ojw28 pushed a commit that referenced this issue Jan 15, 2019
We currently forget whether a source is seekable at re-preparation. This was
implemented intentionally this way under the assumption that we really can't seek
until we have loaded the seek map again. However, seek operations are only
allowed after a media period is prepared. So there is no harm in remembering
whether a source is seekable.

This problem currently prevents reusing ClippingMediaSources with
ExtractorMediaSource and a non-zero start clip position.

Issue: #5351
PiperOrigin-RevId: 229169441
@tonihei
Copy link
Collaborator

tonihei commented Jan 18, 2019

Closing as fixed.

@tonihei tonihei closed this as completed Jan 18, 2019
@google google locked and limited conversation to collaborators Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants