Skip to content
Permalink
Browse files

Auto merge of #22995 - servo:jdm-patch-28, r=nox

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 db29cb01b0450c29d0ac45c6de2819a4f87e400a
Showing with 0 additions and 1 deletion.
  1. +0 −1 components/script/dom/document.rs
@@ -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

0 comments on commit db29cb0

Please sign in to comment.
You can’t perform that action at this time.