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

Merged
merged 6 commits into from Apr 28, 2020
Merged

Conversation

@moz-gfx
Copy link

moz-gfx commented Apr 27, 2020

No description provided.

…ions. r=kats,Bert

They aren't used, so can easily be removed as the first part of
this seried of patches.

If this functionality is ever required, it can be handled by the
caller defining complex clip nodes explicitly.

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

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

moz-gfx commented Apr 27, 2020

@bors-servo r=auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2020

📌 Commit 2057464 has been approved by auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2020

Testing commit 2057464 with merge af6ca76...

bors-servo added a commit that referenced this pull request Apr 27, 2020
Sync changes from mozilla-central gfx/wr
@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2020

💔 Test failed - status-appveyor

@staktrace
Copy link
Contributor

staktrace commented Apr 27, 2020

@staktrace
Copy link
Contributor

staktrace commented Apr 27, 2020

@bors-servo r=auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2020

📌 Commit 2057464 has been approved by auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2020

Testing commit 2057464 with merge 1e07687...

bors-servo added a commit that referenced this pull request Apr 27, 2020
Sync changes from mozilla-central gfx/wr
gw3583 added 2 commits Apr 27, 2020
This allows defining rectangle clips directly, rather than via
the generic clip region methods.

This patch doesn't make use of it in Gecko yet, but it will do
once we add the remaining rounded rect API.

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

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/eed6e488f1e431850c67e960e7c8cd30ff5fa1d3
This adds the last remaining explicit API for defining clips of
a specific type, and ports wrench and examples to use them.

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

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

moz-gfx commented Apr 27, 2020

@bors-servo r=auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2020

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

  • This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.
@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2020

📌 Commit 8a4b784 has been approved by auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2020

Testing commit 8a4b784 with merge 61cf7be...

bors-servo added a commit that referenced this pull request Apr 27, 2020
Sync changes from mozilla-central gfx/wr
@bors-servo
Copy link
Contributor

bors-servo commented Apr 28, 2020

💔 Test failed - status-taskcluster

jamienicol and others added 3 commits Apr 28, 2020
…he frame being a no-op. r=gw

The RenderBackend can be sent an `invalidate_rendered_frame` flag to
indicate that the current rendered frame is invalid. This is useful
when the platform requires a render, eg when starting or resuming the
app on Android. Upon receiving this flag, the render backend will set
a variable `doc.rendered_frame_is_valid = false`. Later on it will
decide to skip rendering only if this variable is true, so by setting
the invalidate flag we should be able to ensure the next render will
occur.

However, the RenderBackend also tries to skip renders which it
determines are not required. Currently it does this by setting
`doc.rendered_frame_is_valid = true` if it decides the frame is a
no-op. This overwrites the previous value set by the
`invalidate_rendered_frame` flag, meaning webrender skips renderering
even though the platform has requested it.

This was resulting in the GVE app showing a black screen on startup,
and Fenix temporarily showing a black screen whilst opening a new tab,
because despite WebRenderBridgeParent requesting an invalidation
immediately on startup, webrender ignored that request until it
decided it actually had content to paint.

To fix this, the logic should be flipped. The value of
`doc.rendered_frame_is_valid` must be remembered across document
updates rather than defaulting to false. And instead of setting it
true if webrender thinks the frame is a no-op, we must set it false if
webrender thinks the frame is *not* a no-op.

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

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

moz-gfx commented Apr 28, 2020

@bors-servo r=auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 28, 2020

📌 Commit 573b4de has been approved by auto

@bors-servo
Copy link
Contributor

bors-servo commented Apr 28, 2020

Testing commit 573b4de with merge 01082a9...

@bors-servo
Copy link
Contributor

bors-servo commented Apr 28, 2020

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

@bors-servo bors-servo merged commit 01082a9 into servo:master Apr 28, 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

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