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

Do not lock if GL thread fails #27440

Closed
wants to merge 1 commit into from
Closed

Conversation

@paulrouget
Copy link
Contributor

paulrouget commented Jul 29, 2020

This might help sometimes when the GL thread fails.

@jdm as I was mentioning, without this, the whole browser can end up being locked (UI being unresponsive).
With this, Servo stays irresponsive, but the app is not locked.

@jdm
Copy link
Member

jdm commented Jul 29, 2020

Ok, this is subtle enough that I'd like to spend a bit of time testing it on the device to make sure there are no surprises. Let's target it for the next release instead of the one for this week.

@jdm jdm added this to the Firefox Reality vNext milestone Jul 29, 2020
@Manishearth Manishearth requested a review from jdm Jul 29, 2020
@Manishearth
Copy link
Member

Manishearth commented Jul 29, 2020

r? @jdm

@highfive highfive assigned jdm and unassigned Manishearth Jul 29, 2020
@paulrouget
Copy link
Contributor Author

paulrouget commented Jul 29, 2020

We can skip that PR if it's not for v1.2, as it will be part of #27438.

@bors-servo
Copy link
Contributor

bors-servo commented Aug 4, 2020

The latest upstream changes (presumably #27438) made this pull request unmergeable. Please resolve the merge conflicts.

@paulrouget
Copy link
Contributor Author

paulrouget commented Aug 5, 2020

This happened in #27438

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

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