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 PrimitiveMetadata struct. #3218
Conversation
This is prep work for changing the structure of the prepare_prims pass. Currently, the code is (roughly): - Traverse primitive tree, recursing through pictures. - When visible, call prepare_prim to update GPU cache, clips etc. - Assign render tasks to passes. - Assign primitive lists for each picture to batches. In future, it will instead be: - Traverse picture tree, but not individual primitives. - Determine surface allocation, plane-splitting, visibility. - For each visible picture, prepare_prims and add to batches. This will simplify some existing code, as well as making possible some requirements for picture caching (e.g. determining the scale to rasterize a picture at before creating the child render tasks).
Try run looks good. |
It doesn't really serve any useful purpose anymore, and it will slightly simplify things as we move to storing primitives as interned / immutable data + prim instances.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 3 of 3 files at r2.
Reviewable status: 3 of 7 files reviewed, all discussions resolved
@bors-servo r+ |
📌 Commit b7ba28b has been approved by |
Remove PrimitiveMetadata struct. It doesn't really serve any useful purpose anymore, and it will slightly simplify things as we move to storing primitives as interned / immutable data + prim instances. <!-- 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/3218) <!-- Reviewable:end -->
💔 Test failed - status-appveyor |
AppVeyor network error again:
Time to migrate the Windows CI to Travis? @bors-servo retry |
Remove PrimitiveMetadata struct. It doesn't really serve any useful purpose anymore, and it will slightly simplify things as we move to storing primitives as interned / immutable data + prim instances. <!-- 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/3218) <!-- Reviewable:end -->
☀️ Test successful - status-appveyor, status-taskcluster |
It doesn't really serve any useful purpose anymore, and it
will slightly simplify things as we move to storing primitives
as interned / immutable data + prim instances.
This change is