Skip to content

Remove some unused support for the legacy network stack in script.#13741

Merged
bors-servo merged 2 commits intomasterfrom
fetch-doc-unused
Oct 13, 2016
Merged

Remove some unused support for the legacy network stack in script.#13741
bors-servo merged 2 commits intomasterfrom
fetch-doc-unused

Conversation

@Ms2ger
Copy link
Copy Markdown
Contributor

@Ms2ger Ms2ger commented Oct 13, 2016

This change is Reviewable

@highfive
Copy link
Copy Markdown

Heads up! This PR modifies the following files:

  • @KiChjang: components/script/script_thread.rs, components/script/dom/document.rs, components/script/document_loader.rs

@highfive
Copy link
Copy Markdown

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 13, 2016
@Ms2ger
Copy link
Copy Markdown
Contributor Author

Ms2ger commented Oct 13, 2016

Reviewed in #13714.

@bors-servo r=Manishearth

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit d0323e3 has been approved by Manishearth

@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Oct 13, 2016
@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit d0323e3 with merge 740464d...

bors-servo pushed a commit that referenced this pull request Oct 13, 2016
Remove some unused support for the legacy network stack in script.

<!-- 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/13741)
<!-- Reviewable:end -->
@bors-servo
Copy link
Copy Markdown
Contributor

@bors-servo bors-servo merged commit d0323e3 into master Oct 13, 2016
@highfive highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 13, 2016
@Ms2ger Ms2ger deleted the fetch-doc-unused branch October 13, 2016 15:43
@Ms2ger Ms2ger mentioned this pull request Nov 7, 2016
31 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants