Skip to content

Commit

Permalink
Auto merge of #22995 - servo:jdm-patch-28, r=nox
Browse files Browse the repository at this point in the history
Remove an unnecessary synchronous full reflow.

We already reflow any nodes that are dirtied during any turn of the event loop. There is no reason to synchronously reflow the entire document, especially when we don't even modify it in this method.

- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #22988
- [x] There are tests for these changes

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/22995)
<!-- Reviewable:end -->
  • Loading branch information
bors-servo committed Mar 8, 2019
2 parents 34a5a82 + a8683bc commit f0c62f4
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion components/script/dom/document.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2201,7 +2201,6 @@ impl Document {
window,
);

window.reflow(ReflowGoal::Full, ReflowReason::DOMContentLoaded);
update_with_current_time_ms(&self.dom_content_loaded_event_end);

// html parsing has finished - set dom content loaded
Expand Down

0 comments on commit f0c62f4

Please sign in to comment.