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 (now unused) nested display list APIs. #1806

Merged
merged 1 commit into from
Oct 5, 2017

Conversation

glennw
Copy link
Member

@glennw glennw commented Oct 4, 2017

This change is Reviewable

@glennw
Copy link
Member Author

glennw commented Oct 4, 2017

r? @mrobinson and @jrmuizel

(Should probably not merge until @jrmuizel confirms that https://bugzilla.mozilla.org/show_bug.cgi?id=1405819 has landed).

It wasn't clear to me whether the different types of ClipIds (external, dynamic etc) are related to nesting. Is that for something else or should they be modified too?

Copy link
Member

@mrobinson mrobinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes me really happy!

@kvark
Copy link
Member

kvark commented Oct 5, 2017

One less thing in WR that I don't fully understand :D

@kvark
Copy link
Member

kvark commented Oct 5, 2017

The Gecko change has landed (with RESOLVED FIXED status).
@bors-servo r=mrobinson

@bors-servo
Copy link
Contributor

📌 Commit b4fbc86 has been approved by mrobinson

@bors-servo
Copy link
Contributor

⌛ Testing commit b4fbc86 with merge a077be4...

bors-servo pushed a commit that referenced this pull request Oct 5, 2017
Remove the (now unused) nested display list APIs.

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

☀️ Test successful - status-appveyor, status-travis
Approved by: mrobinson
Pushing a077be4 to master...

@bors-servo bors-servo merged commit b4fbc86 into servo:master Oct 5, 2017
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Oct 5, 2017
Bug 1405819 removed the last usage, and this will be needed for
servo/webrender#1806
JerryShih pushed a commit to JerryShih/gecko-dev that referenced this pull request Oct 9, 2017
Bug 1405819 removed the last usage, and this will be needed for
servo/webrender#1806
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 2, 2019
Bug 1405819 removed the last usage, and this will be needed for
servo/webrender#1806

UltraBlame original commit: 8544b65923a370aaed46a36699349d889e591fe1
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 2, 2019
Bug 1405819 removed the last usage, and this will be needed for
servo/webrender#1806

UltraBlame original commit: 8544b65923a370aaed46a36699349d889e591fe1
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 2, 2019
Bug 1405819 removed the last usage, and this will be needed for
servo/webrender#1806

UltraBlame original commit: 8544b65923a370aaed46a36699349d889e591fe1
github-actions bot pushed a commit to declantsien/webrender-native that referenced this pull request Jan 26, 2025
Bug 1405819 removed the last usage, and this will be needed for
servo/webrender#1806
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.

5 participants