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

Inverted composition disabled flag to be composition enabled #1172

Merged
merged 1 commit into from
Feb 8, 2021

Conversation

toji
Copy link
Member

@toji toji commented Feb 8, 2021

Fixes #1171

The fact that this was phrased as "disabled" was a historical artifact left over from a point when the flag was going to be exposed as part of the API surface and we wanted it to default to false.


Preview | Diff

@toji toji requested a review from Manishearth February 8, 2021 18:55
@toji toji merged commit 1a70fd1 into main Feb 8, 2021
webkit-commit-queue pushed a commit to WebKit/WebKit that referenced this pull request Feb 9, 2021
https://bugs.webkit.org/show_bug.cgi?id=221599

Patch by Imanol Fernandez <ifernandez@igalia.com> on 2021-02-09
Reviewed by Sergio Villar Senin.

The spec agreed to rename this to avoid double negations in code.
See immersive-web/webxr#1172.

* Modules/webxr/WebXRWebGLLayer.cpp:
(WebCore::WebXRWebGLLayer::create):
(WebCore::WebXRWebGLLayer::WebXRWebGLLayer):
* Modules/webxr/WebXRWebGLLayer.h:

Canonical link: https://commits.webkit.org/233843@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@272571 268f45cc-cd09-0410-ab3c-d52691b4dbfc
bertogg pushed a commit to Igalia/webkit that referenced this pull request Feb 9, 2021
https://bugs.webkit.org/show_bug.cgi?id=221599

Patch by Imanol Fernandez <ifernandez@igalia.com> on 2021-02-09
Reviewed by Sergio Villar Senin.

The spec agreed to rename this to avoid double negations in code.
See immersive-web/webxr#1172.

* Modules/webxr/WebXRWebGLLayer.cpp:
(WebCore::WebXRWebGLLayer::create):
(WebCore::WebXRWebGLLayer::WebXRWebGLLayer):
* Modules/webxr/WebXRWebGLLayer.h:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@272571 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider renaming "composition disabled" to "composition enabled" flag
2 participants