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

stylo: Use AtomicRefCell for PerDocumentStyleData #13688

Merged
merged 1 commit into from Oct 11, 2016

Conversation

@bholley
Copy link
Contributor

bholley commented Oct 11, 2016

@highfive
Copy link

highfive commented Oct 11, 2016

warning Warning warning

  • These commits modify style code, but no tests are modified. Please consider adding a test!
@bholley
Copy link
Contributor Author

bholley commented Oct 11, 2016

@bors-servo r=Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Oct 11, 2016

📌 Commit a2212f8 has been approved by Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Oct 11, 2016

Testing commit a2212f8 with merge 1c56dca...

bors-servo added a commit that referenced this pull request Oct 11, 2016
 stylo: Use AtomicRefCell for PerDocumentStyleData

Gecko bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1309090

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

bors-servo commented Oct 11, 2016

💔 Test failed - mac-rel-wpt2

@Manishearth
Copy link
Member

Manishearth commented Oct 11, 2016

@nox nox added S-awaiting-merge and removed S-tests-failed labels Oct 11, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Oct 11, 2016

Testing commit a2212f8 with merge 2a4bb9d...

bors-servo added a commit that referenced this pull request Oct 11, 2016
 stylo: Use AtomicRefCell for PerDocumentStyleData

Gecko bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1309090

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

bors-servo commented Oct 11, 2016

💔 Test failed - linux-dev

@bholley bholley force-pushed the bholley:styleset_refcell branch from a2212f8 to 1b3a932 Oct 11, 2016
MozReview-Commit-ID: 27JeRluDAo3
@bholley bholley force-pushed the bholley:styleset_refcell branch from 1b3a932 to 63ac0d2 Oct 11, 2016
@bholley
Copy link
Contributor Author

bholley commented Oct 11, 2016

@bors-servo r=Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Oct 11, 2016

📌 Commit 63ac0d2 has been approved by Manishearth

@bholley
Copy link
Contributor Author

bholley commented Oct 11, 2016

@Manishearth thanks for doing #13620! Already paid off in this PR, where I forgot to update the signatures of the Servo_* functions in glue.rs. Testing and infra FTW!

@Manishearth
Copy link
Member

Manishearth commented Oct 11, 2016

😄

@bors-servo
Copy link
Contributor

bors-servo commented Oct 11, 2016

Testing commit 63ac0d2 with merge 3916d1d...

bors-servo added a commit that referenced this pull request Oct 11, 2016
 stylo: Use AtomicRefCell for PerDocumentStyleData

Gecko bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1309090

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

bors-servo commented Oct 11, 2016

@bors-servo bors-servo merged commit 63ac0d2 into servo:master Oct 11, 2016
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test successful
Details
@bholley bholley deleted the bholley:styleset_refcell branch Oct 30, 2016
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.

None yet

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