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

[WPE][GTK] Build broken with ENABLE_VIDEO=OFF after r292252 #1179

Conversation

aperezdc
Copy link
Contributor

@aperezdc aperezdc commented May 31, 2022

4fd8a09

[WPE][GTK] Build broken with ENABLE_VIDEO=OFF after r292252
https://bugs.webkit.org/show_bug.cgi?id=241123

Reviewed by Philippe Normand.

* Source/WebKit/WebProcess/WebPage/WebPage.cpp: Add missing ENABLE(VIDEO) guard.
* Source/WebKit/WebProcess/WebPage/WebPage.h: Ditto.

Canonical link: https://commits.webkit.org/251142@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@295047 268f45cc-cd09-0410-ab3c-d52691b4dbfc

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

Patch by Žan Doberšek <zdobersek@igalia.com> on 2022-05-31
Reviewed by Miguel Gomez and Philippe Normand.

* Source/WebCore/platform/graphics/gstreamer/DMABufVideoSinkGStreamer.cpp:
NV12 is listed twice in the DMABufVideoSink's list of supported formats. This
is a typo, one of those was supposed to be NV21, a different-but-similar format
that works just fine.

Canonical link: https://commits.webkit.org/251141@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@295046 268f45cc-cd09-0410-ab3c-d52691b4dbfc
@aperezdc aperezdc self-assigned this May 31, 2022
@aperezdc aperezdc added Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases WebKit Nightly Build labels May 31, 2022
@aperezdc aperezdc added the merge-queue Applied to send a pull request to merge-queue label May 31, 2022
https://bugs.webkit.org/show_bug.cgi?id=241123

Reviewed by Philippe Normand.

* Source/WebKit/WebProcess/WebPage/WebPage.cpp: Add missing ENABLE(VIDEO) guard.
* Source/WebKit/WebProcess/WebPage/WebPage.h: Ditto.

Canonical link: https://commits.webkit.org/251142@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@295047 268f45cc-cd09-0410-ab3c-d52691b4dbfc
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/WPEGTK-Build-broken-with-ENABLE_VIDEOOFF-after-r292252 branch from e9ce1a6 to 4fd8a09 Compare May 31, 2022 12:33
@webkit-early-warning-system
Copy link
Collaborator

Committed r295047 (251142@main): https://commits.webkit.org/251142@main

Reviewed commits have been landed. Closing PR #1179 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system removed the merge-queue Applied to send a pull request to merge-queue label May 31, 2022
@aperezdc aperezdc deleted the eng/WPEGTK-Build-broken-with-ENABLE_VIDEOOFF-after-r292252 branch May 31, 2022 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases
Projects
None yet
4 participants