Skip to content

/ servo

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

Jump to bottom

Reimplement the COW DOM #447

Closed
brson opened this issue May 14, 2013 · 4 comments
Closed

Reimplement the COW DOM #447

brson opened this issue May 14, 2013 · 4 comments

Comments

@brson
Copy link
Contributor

@brson brson commented May 14, 2013

Servo had a copy-on-write DOM for a while, but it was complicated and we removed it. Now we want it again, but have some different ideas about the implementation.

TODO: Outline design.
@bzbarsky
Copy link
Contributor

@bzbarsky bzbarsky commented May 14, 2013

Is there any publicly available discussion, or at least description of the ideas?

Measuring the performance of this stuff to validate the design is, imo, is a much higher priority than a lot of the other stuff we seem to be doing (e.g. border painting stubs).
@brson
Copy link
Contributor Author

@brson brson commented May 14, 2013

@bzbarsky No, I do not think there is anything written down on this subject.
@pcwalton
Copy link
Contributor

@pcwalton pcwalton commented May 15, 2013

@bzbarsky I just filed issue #449 for some performance benchmarking.

The design hasn't really changed since the last mailing list posts. There was a more in-depth presentation that I gave a couple of times in Vancouver. @metajack took notes and is planning to write them up I believe.
@jdm
Copy link
Member

@jdm jdm commented Feb 4, 2015

We've punted on the COW DOM.
@jdm jdm closed this Feb 4, 2015
glennw pushed a commit to glennw/servo that referenced this issue Jan 16, 2017
@bors-servo
Auto merge of servo#447 - glennw:fix-437, r=mrobinson
1070d23 
Fix shader compile error for image + clip + transform.

Closes servo#437.

<!-- 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/447)
<!-- Reviewable:end -->
@jdm jdm mentioned this issue Nov 5, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@jdm @brson @pcwalton @bzbarsky
You can’t perform that action at this time.