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

[MSE][GStreamer] Append pipeline should accept out of segment buffers #4414

Closed
wants to merge 1 commit into from

Conversation

calvaris
Copy link
Contributor

@calvaris calvaris commented Sep 16, 2022

39adafe

[MSE][GStreamer] Append pipeline should accept out of segment buffers
https://bugs.webkit.org/show_bug.cgi?id=245272

Reviewed by NOBODY (OOPS!).

In complex streams (combining commercials and content, for example) the demuxer
could output out of clip segments that will be corrected with timestamp offset
later.

* Source/WebCore/platform/graphics/gstreamer/mse/AppendPipeline.cpp:
(WebCore::AppendPipeline::Track::initializeElements):

39adafe

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios   πŸ›  mac   πŸ›  wpe βœ… πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim   πŸ›  mac-debug   πŸ›  gtk   πŸ›  wincairo
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2   πŸ›  mac-AS-debug   πŸ§ͺ gtk-wk2
  πŸ§ͺ api-ios   πŸ§ͺ api-mac   πŸ§ͺ api-gtk
βœ… πŸ›  tv   πŸ§ͺ mac-wk1
βœ… πŸ›  tv-sim   πŸ§ͺ mac-wk2
βœ… πŸ›  watch   πŸ§ͺ mac-AS-debug-wk2
βœ… πŸ›  watch-sim   πŸ§ͺ mac-wk2-stress

https://bugs.webkit.org/show_bug.cgi?id=245272

Reviewed by NOBODY (OOPS!).

In complex streams (combining commercials and content, for example) the demuxer
could output out of clip segments that will be corrected with timestamp offset
later.

* Source/WebCore/platform/graphics/gstreamer/mse/AppendPipeline.cpp:
(WebCore::AppendPipeline::Track::initializeElements):
@calvaris calvaris self-assigned this Sep 16, 2022
@calvaris calvaris added Media Bugs related to the HTML 5 Media elements. WebKit Nightly Build labels Sep 16, 2022
@calvaris
Copy link
Contributor Author

I was lousy and did not check that this code was already included two lines above.

@calvaris calvaris closed this Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Media Bugs related to the HTML 5 Media elements.
Projects
None yet
2 participants