-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
layout: Don't panic if requestAnimationFrame()
is called before first layout.
#7294
Conversation
@bors-servo r=glennw r+ Fixed the test. |
📌 Commit 575047c has been approved by |
📌 Commit 575047c has been approved by |
…ennw layout: Don't panic if `requestAnimationFrame()` is called before first layout. Closes #7115. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7294) <!-- Reviewable:end -->
💔 Test failed - mac1 |
self.profiler_metadata(), | ||
self.time_profiler_chan.clone(), | ||
|| animation::recalc_style_for_animations(root_flow.deref_mut(), animations)); | ||
match rw_data.root_flow.as_ref() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if let Some(ref root_flow) =
?
Gah, looks like rAF just doesn't work headless. |
…st layout. Closes #7115.
575047c
to
a516042
Compare
@bors-servo r=glennw+Ms2ger |
📌 Commit a516042 has been approved by |
…ennw+Ms2ger layout: Don't panic if `requestAnimationFrame()` is called before first layout. Closes #7115. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7294) <!-- Reviewable:end -->
Closes #7115.