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

Sync changes from mozilla-central gfx/wr #4848

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

moz-gfx
Copy link

@moz-gfx moz-gfx commented Mar 13, 2024

No description provided.

nical and others added 5 commits March 12, 2024 09:57
…or Gecko r=jrmuizel

Make glean an optional dependency that is only enabled when building
WebRender for Gecko.

Differential Revision: https://phabricator.services.mozilla.com/D204059

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/10af6b2ff03bd7512d53f3ed02f7e83dba7c57ba
@moz-gfx
Copy link
Author

moz-gfx commented Mar 13, 2024

@bors-servo r=auto

@bors-servo
Copy link
Contributor

📌 Commit 944d370 has been approved by auto

@bors-servo
Copy link
Contributor

🔒 Merge conflict

nical and others added 5 commits March 13, 2024 21:40
…ct. r=gfx-reviewers,gw

While investigating why get_surface_rect sometimes returns an empty rect (causing bad render task sizes), I noticed that all users of get_surface_rect use it to size surfaces, rely on the returned rect to be snapped to integer coordinates, and some even round the coordinates a second time. The documentation of the function states that its purpose is to provide a rect suitable for surface allocation. This patch bakes the expectation into the type and removes some redundant operations.

Differential Revision: https://phabricator.services.mozilla.com/D203738

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/eae4f8752721b2e1361c4c1ea57e038a01e557ea
…ausing webrender bustages in conftest.c CLOSED TREE

Backed out changeset 987284103c26 (bug 1769492)
Backed out changeset dd1b3d97c6a8 (bug 1769492)
Backed out changeset 4effb464f604 (bug 1883863)
Backed out changeset eae4f8752721 (bug 1883873)

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/d9ad55d66d8523561815b7fbeef13f84428eb55b
@moz-gfx
Copy link
Author

moz-gfx commented Mar 13, 2024

@bors-servo r=auto

@bors-servo
Copy link
Contributor

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.

@bors-servo
Copy link
Contributor

📌 Commit 944d370 has been approved by auto

The vClipRadii varying being changed to mediump is causing borders to
be clipped incorrectly at certain zoom levels. This patch reverts it
to highp to avoid the issue.

As a precaution it conservatively reverts several other varyings in
this shader to be highp as well. Although the others are not
responsible for the issue reported in this bug, using mediump for
varyings related to positioning could cause similar problems.

Differential Revision: https://phabricator.services.mozilla.com/D204507

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/7afa472a8cac8d854d0d9c68a3c18a590978fffe
@moz-gfx
Copy link
Author

moz-gfx commented Mar 14, 2024

@bors-servo r=auto

@bors-servo
Copy link
Contributor

📌 Commit 6e4ff49 has been approved by auto

nical and others added 5 commits March 17, 2024 23:22
…ct. r=gfx-reviewers,gw

While investigating why get_surface_rect sometimes returns an empty rect (causing bad render task sizes), I noticed that all users of get_surface_rect use it to size surfaces, rely on the returned rect to be snapped to integer coordinates, and some even round the coordinates a second time. The documentation of the function states that its purpose is to provide a rect suitable for surface allocation. This patch bakes the expectation into the type and removes some redundant operations.

Differential Revision: https://phabricator.services.mozilla.com/D203738

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/038ff8035b29b9052311d9cf616a2aced886896c
Webrender's glslopt-optimized shaders encounter a miscompilation on
some Adreno driver versions regarding fetching empty clip tasks. This
patch reshuffles the code in such a way as to avoid the
bug. Unfortunately the specific cause of the miscompilation remains
unknown, meaning we must take extra care not to regress it in the
future.

Differential Revision: https://phabricator.services.mozilla.com/D204864

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/488c57bb8ceed6da3f0be935156a480cdb92bc32
@moz-gfx
Copy link
Author

moz-gfx commented Mar 17, 2024

@bors-servo r=auto

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.

None yet

6 participants