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 #3919

Merged
merged 4 commits into from Apr 15, 2020
Merged

Conversation

@moz-gfx
Copy link

moz-gfx commented Apr 15, 2020

No description provided.

Glenn Watson and others added 4 commits Apr 15, 2020
Only drop targets from the render target pool when the size of
the pool is larger than an arbitrary threshold (this is 32 MB
for now), _and_ the render target hasn't been used in the last
60 frames of rendering.

This reduces the number of allocation thrashing of textures in
the render target pool on most pages.

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

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/c6ed30c910c03820a01baec4dd7e4884e8e37cac
These just wrap regular std Sender/Receiver without providing any value. Serialize/Deserialize was implement manually for MsgSender/MsgReceiver to assert. Serde being amazing, it provides with annotations to not require the traits to be implemented on some enum variants and assert at runtime which functionally equivalent but less error-prone than the fake trait implementations.

Removing the rest of channel.rs is coming in a followup.

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

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/b0d79edece1314cc5962c1f625bdffb947419425
…n adreno r=kvark

Previously we were using 256 bytes (which happens to be equal to 64
pixels for RGBA8 textures). A recent change to the GPU Cache uncovered
the fact that the requirement is actually 64 pixels, eg 1024 bytes for
RGBAF32 textures.

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

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

moz-gfx commented Apr 15, 2020

@bors-servo r=auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 15, 2020

📌 Commit 13e3bfd has been approved by auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 15, 2020

Testing commit 13e3bfd with merge f5db59b...

@bors-servo
Copy link
Contributor

bors-servo commented Apr 15, 2020

☀️ Test successful - status-appveyor, status-taskcluster
Approved by: auto
Pushing f5db59b to master...

@bors-servo bors-servo merged commit f5db59b into servo:master Apr 15, 2020
3 checks passed
3 checks passed
Community-TC (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.