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

Remove all old layers before adding a new root layer (fixes #2259) #2575

Merged
merged 3 commits into from Jun 4, 2014

Conversation

mbrubeck
Copy link
Contributor

@mbrubeck mbrubeck commented Jun 4, 2014

This fixes #2259 which was caused by some of the previous layers sticking around incorrectly. Depends on servo/rust-layers#69.

Also includes some related cleanup as separate commits.

No reftest because our test harness doesn't yet support navigating to a new page (#2574).

This isn't related to any known issue, but is just to ensure that the IDs are
set correctly within this function.
@hoppipolla-critic-bot
Copy link

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

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.

bors-servo pushed a commit that referenced this pull request Jun 4, 2014
This fixes #2259 which was caused by some of the previous layers sticking around incorrectly.  Depends on servo/rust-layers#69.

Also includes some related cleanup as separate commits.

No reftest because our test harness doesn't yet support navigating to a new page (#2574).
@bors-servo bors-servo closed this Jun 4, 2014
@bors-servo bors-servo merged commit 81601c2 into servo:master Jun 4, 2014
@mbrubeck mbrubeck deleted the 2259-remove-old-layers branch June 24, 2014 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New page painted on top of old page when navigating
4 participants