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

Remove the unused StackingContext has_stacking_contexts field #518

Merged
merged 1 commit into from Nov 3, 2016

Conversation

@mrobinson
Copy link
Member

mrobinson commented Nov 3, 2016

It seems that this field has been unused since the switch to WR2.


This change is Reviewable

It seems that this field has been unused since the switch to WR2.
@mrobinson
Copy link
Member Author

mrobinson commented Nov 3, 2016

@glennw r?

@glennw
Copy link
Member

glennw commented Nov 3, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Nov 3, 2016

📌 Commit 29f99df has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Nov 3, 2016

Testing commit 29f99df with merge f277305...

bors-servo added a commit that referenced this pull request Nov 3, 2016
Remove the unused StackingContext has_stacking_contexts field

It seems that this field has been unused since the switch to WR2.

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

bors-servo commented Nov 3, 2016

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit 29f99df into servo:master Nov 3, 2016
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@mrobinson mrobinson deleted the mrobinson:remove-unused-field branch Nov 16, 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

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