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

Refactor compositor layer tree design #2748

Merged
merged 1 commit into from Jul 2, 2014
Merged

Conversation

@mrobinson
Copy link
Member

mrobinson commented Jul 2, 2014

Instead of having two parallel trees of CompositorLayers and
ContainerLayers, transform CompositorLayer to CompositorData and move
tiling logic to rust-layers.

Instead of having two parallel trees of CompositorLayers and
ContainerLayers, transform CompositorLayer to CompositorData and move
tiling logic to rust-layers.
@hoppipolla-critic-bot
Copy link

hoppipolla-critic-bot commented Jul 2, 2014

Critic review: https://critic.hoppipolla.co.uk/r/1950

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Jul 2, 2014

@zwarich
Copy link

zwarich commented Jul 2, 2014

Just adding a note that the corresponding rust-layers change was done in servo/rust-layers#75.

@larsbergstrom

This comment has been minimized.

Copy link

larsbergstrom commented on 0c2538d Jul 2, 2014

r=zwarich

larsbergstrom added a commit that referenced this pull request Jul 2, 2014
Refactor compositor layer tree design
@larsbergstrom larsbergstrom merged commit 0a910ce into servo:master Jul 2, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
@mrobinson mrobinson deleted the mrobinson:layer-reorg branch Jul 2, 2014
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

4 participants
You can’t perform that action at this time.