Skip to content
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

Don't load stylesheets in documents without browsing contexts #12241

Merged
merged 2 commits into from Jul 5, 2016

Conversation

@jdm
Copy link
Member

jdm commented Jul 4, 2016

Per whatwg/html#1495, this fixes the most commonly reported panic in nightlies by making us match what other browsers do.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #12047 (github issue number if applicable).
  • There are tests for these changes OR

This change is Reviewable

@highfive
Copy link

highfive commented Jul 4, 2016

Heads up! This PR modifies the following files:

  • @KiChjang: components/script/dom/htmllinkelement.rs
@jdm jdm force-pushed the jdm:multiple-stylesheets branch from 8bfb7bf to 7b40c3b Jul 4, 2016
@notriddle
Copy link
Contributor

notriddle commented Jul 5, 2016

Are you sure 1495, a long-closed bug about heap corruption on Android, is related to this?

@jdm
Copy link
Member Author

jdm commented Jul 5, 2016

My mistake, I meant whatwg/html#1495.

@notriddle
Copy link
Contributor

notriddle commented Jul 5, 2016

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@cbrewster
Copy link
Member

cbrewster commented Jul 5, 2016

@bors-servo delegate=notriddle

@bors-servo
Copy link
Contributor

bors-servo commented Jul 5, 2016

✌️ @notriddle can now approve this pull request

@notriddle
Copy link
Contributor

notriddle commented Jul 5, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jul 5, 2016

📌 Commit a840717 has been approved by notriddle

@bors-servo
Copy link
Contributor

bors-servo commented Jul 5, 2016

Testing commit a840717 with merge 6149144...

bors-servo added a commit that referenced this pull request Jul 5, 2016
Don't load stylesheets in documents without browsing contexts

Per whatwg/html#1495, this fixes the most commonly reported panic in nightlies by making us match what other browsers do.

---
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #12047 (github issue number if applicable).
- [X] There are tests for these changes OR

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12241)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 5, 2016

@bors-servo bors-servo merged commit a840717 into servo:master Jul 5, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@Ms2ger Ms2ger deleted the jdm:multiple-stylesheets branch Jul 5, 2016
@asajeffrey asajeffrey mentioned this pull request Sep 6, 2019
4 of 4 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

6 participants
You can’t perform that action at this time.