-
-
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
Add cancellability to file manager load and related refactoring #12684
Conversation
Heads up! This PR modifies the following files:
|
Another patch for mis-design :/ |
☔ The latest upstream changes (presumably #12579) made this pull request unmergeable. Please resolve the merge conflicts. |
@bors-servo r+ Don't worry about the mis-design, happens all the time 😄 |
📌 Commit c2215ca has been approved by |
@bors-servo r- needs rebase |
done |
@bors-servo r=Manishearth |
📌 Commit 17ae38a has been approved by |
Add cancellability to file manager load and related refactoring Fixes #12589. r? @Manishearth --- - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #12589 - [x] There are tests for these changes OR <!-- 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/12684) <!-- Reviewable:end -->
💔 Test failed - mac-rel-css |
|
⚡ Previous build results for arm32, arm64, linux-dev, mac-dev-unit, windows-dev are reusable. Rebuilding only linux-rel, mac-rel-css, mac-rel-wpt... |
☀️ Test successful - arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev |
Fixes #12589.
r? @Manishearth
./mach build -d
does not report any errors./mach test-tidy
does not report any errorsThis change is