Skip to content

Update webrender for latest RenderNotifier changes#18976

Closed
jdm wants to merge 1 commit intoservo:masterfrom
jdm:wr-notifier
Closed

Update webrender for latest RenderNotifier changes#18976
jdm wants to merge 1 commit intoservo:masterfrom
jdm:wr-notifier

Conversation

@jdm
Copy link
Member

@jdm jdm commented Oct 20, 2017

This addresses one of the most common intermittent test failures by making it impossible for the failure to occur. Requires servo/webrender#1904 to be merged upstream first.


This change is Reviewable

@jdm jdm added the S-blocked-on-external Something, somewhere else, needs to happen before this PR can be merged. label Oct 20, 2017
@highfive
Copy link

Heads up! This PR modifies the following files:

  • @paulrouget: components/compositing/lib.rs, components/compositing/compositor_thread.rs, components/servo/lib.rs, components/compositing/compositor.rs

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 20, 2017
@jdm
Copy link
Member Author

jdm commented Oct 20, 2017

r? @glennw

@glennw
Copy link
Member

glennw commented Oct 23, 2017

@jdm I included this commit of yours in #18986 - feel free to either r+ that PR, or close that and apply the changes to this PR.

@jdm jdm closed this Oct 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed. S-blocked-on-external Something, somewhere else, needs to happen before this PR can be merged.

Projects

None yet

4 participants