Skip to content

Conversation

moz-gfx
Copy link

@moz-gfx moz-gfx commented Apr 1, 2023

No description provided.

… control the processing of ResultMsg messages r=gfx-reviewers,lsalzman

It is a preparation for Bug 1804233. From bug 1776885 comment 6, we need to defer WebRender rendering on render thread if remote texture is not ready. To do it, ResultMsg messages handling needs to be controlled.

By adding the followings, ResultMsg messages handling could be controlled by RenderThread.
-  FramePublishId to new_frame_ready()
-  Renderer::set_target_frame_publish_id()

new_frame_ready() has frame_publish_id. The publish_id could be set to Renderer::set_target_frame_publish_id(). Then Renderer::update() defers processing of ResultMsg, if frame_publish_id of ResultMsg::PublishDocument exceeds target_frame_publish_id.

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

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

moz-gfx commented Apr 1, 2023

@bors-servo r=auto

@bors-servo
Copy link
Contributor

📌 Commit 4756086 has been approved by auto

@bors-servo
Copy link
Contributor

⌛ Testing commit 4756086 with merge c1cf1ea...

@bors-servo
Copy link
Contributor

☀️ Test successful - status-taskcluster
Approved by: auto
Pushing c1cf1ea to master...

@bors-servo bors-servo merged commit c1cf1ea into servo:master Apr 1, 2023
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.

2 participants