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
Cache which capabilities are enabled in the context (fixes #20534) #21246
Conversation
Heads up! This PR modifies the following files:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -2192,15 +2185,15 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { | |||
// https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.3 | |||
// FIXME: https://github.com/servo/servo/issues/20534 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the FIXME?
Oh, I hadn't seen the comment above, and @nox asked me to review on IRC. Sorry for stealing the review :( |
@bors-servo r=emilio I fixed the FIXMEs. |
📌 Commit 5a9ad01 has been approved by |
Cache which capabilities are enabled in the context (fixes #20534) <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21246) <!-- Reviewable:end -->
☀️ Test successful - android, android-x86, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
This change is