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.

Sign up for GitHub

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 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

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 commented May 14, 2013

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

@pcwalton
Copy link
Contributor

@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 commented Feb 4, 2015

We've punted on the COW DOM.

@jdm jdm closed this as completed Feb 4, 2015
glennw pushed a commit to glennw/servo that referenced this issue Jan 16, 2017
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
jdm pushed a commit to jdm/servo that referenced this issue Mar 6, 2021
Auto merge of servo#447 - paulrouget:initonce, r=jdm
5baf536 
Introduce feature to only allow initialization once.

See servo#22039

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-mozjs/447)
<!-- Reviewable:end -->
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
Development

No branches or pull requests
4 participants
@jdm @brson @pcwalton @bzbarsky