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

Enable 'immediate-output' on streaming through westerossink #1136

Conversation

varumugam123
Copy link

This commit brings back (part of) the change originally made in f9377b1 but removed inadvertently by 0035d13

@modeveci modeveci self-requested a review August 3, 2023 08:33
Copy link

@modeveci modeveci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a custom property needed for westeros-sink. It has been reverted by mistake during decoder/sink updates.

@eocanha
Copy link
Member

eocanha commented Aug 8, 2023

Patch submitted upstream for review as https://bugs.webkit.org/show_bug.cgi?id=259930 / WebKit/WebKit#16479

webkit-commit-queue pushed a commit to eocanha/WebKit that referenced this pull request Aug 9, 2023
…ming through westerossink

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

Reviewed by Philippe Normand.

https://bugs.webkit.org/show_bug.cgi?id=259357 removed the configuration of the "immediate-output" property for westeros sink, but it's still needed for mediastream content.

Original author: Vivek.A <Vivek_Arumugam@comcast.com>
See: WebPlatformForEmbedded/WPEWebKit#1136

* Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
(WebCore::MediaPlayerPrivateGStreamer::configureElementPlatformQuirks): Configure the immediate-output property for westerossink.

Canonical link: https://commits.webkit.org/266720@main
eocanha added a commit that referenced this pull request Aug 9, 2023
…ming through westerossink

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

Reviewed by Philippe Normand.

https://bugs.webkit.org/show_bug.cgi?id=259357 removed the configuration of the "immediate-output" property for westeros sink, but it's still needed for mediastream content.

Original author: Vivek.A <Vivek_Arumugam@comcast.com>
See: #1136

* Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
(WebCore::MediaPlayerPrivateGStreamer::configureElementPlatformQuirks): Configure the immediate-output property for westerossink.

Canonical link: https://commits.webkit.org/266720@main
@eocanha
Copy link
Member

eocanha commented Aug 9, 2023

Patch backported to https://github.com/WebPlatformForEmbedded/WPEWebKit/tree/wpe-2.38 as a4751cc, closing PR.

@eocanha eocanha closed this Aug 11, 2023
carlosgcampos pushed a commit to WebKit/WebKit that referenced this pull request Aug 18, 2023
…gi?id=259930

    [GStreamer][MediaStream][Westeros] Enable 'immediate-output' on streaming through westerossink
    https://bugs.webkit.org/show_bug.cgi?id=259930

    Reviewed by Philippe Normand.

    https://bugs.webkit.org/show_bug.cgi?id=259357 removed the configuration of the "immediate-output" property for westeros sink, but it's still needed for mediastream content.

    Original author: Vivek.A <Vivek_Arumugam@comcast.com>
    See: WebPlatformForEmbedded/WPEWebKit#1136

    * Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
    (WebCore::MediaPlayerPrivateGStreamer::configureElementPlatformQuirks): Configure the immediate-output property for westerossink.

    Canonical link: https://commits.webkit.org/266720@main
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream Related to an upstream bug (or should be at some point) wpe-2.38
Development

Successfully merging this pull request may close these issues.

None yet

3 participants