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

Switched the mask generator to work on tiles #569

Merged
merged 2 commits into from Nov 18, 2016
Merged

Conversation

@kvark
Copy link
Member

kvark commented Nov 17, 2016

Adds a unique tile ID to the compile context, alpha batch tasks, only to end up in the mask task key.
Computes an estimated inner rectangle of the mask intersection, avoids any work for tiles completely inside it.

This change is Reviewable

@kvark kvark force-pushed the kvark:mask_tile branch from 82b5575 to 9fbaa8e Nov 17, 2016
@kvark kvark changed the title Switched the mask generator to work per-tile Switched the mask generator to work on tiles Nov 17, 2016
@kvark kvark changed the title Switched the mask generator to work on tiles [WIP] Switched the mask generator to work on tiles Nov 17, 2016
@kvark kvark force-pushed the kvark:mask_tile branch from d4a19a5 to 9e1340d Nov 17, 2016
@kvark kvark changed the title [WIP] Switched the mask generator to work on tiles Switched the mask generator to work on tiles Nov 17, 2016
@kvark kvark force-pushed the kvark:mask_tile branch from 9e1340d to c9a9475 Nov 17, 2016
@glennw
Copy link
Member

glennw commented Nov 18, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Nov 18, 2016

📌 Commit c9a9475 has been approved by glennw

@glennw glennw merged commit 6f6ba15 into servo:master Nov 18, 2016
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@kvark kvark deleted the kvark:mask_tile branch Nov 18, 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.