Skip to content

/ servo

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

Make Document store Ranges, and update ranges in Node subclasses #6702

Closed
wants to merge 3 commits into from
Closed

Make Document store Ranges, and update ranges in Node subclasses #6702

wants to merge 3 commits into from

Conversation

@dzbarsky
Copy link
Member

dzbarsky commented Jul 22, 2015

Review on Reviewable
@dzbarsky
Implement Range#extractContents
Loading status checks…
41f29d1
@dzbarsky
Copy link
Member Author

dzbarsky commented Jul 22, 2015

Depends on extractContents and I will make some changes after talking to @nox
@dzbarsky dzbarsky force-pushed the dzbarsky:rangeobserver branch 2 times, most recently from e3cf60f to 9374b9b Jul 22, 2015
dzbarsky added 2 commits Jul 15, 2015
@dzbarsky
Make Document store Ranges, and update ranges in Node subclasses
a058529
@dzbarsky
Make sure Range#insertNode calls Node::remove and updates range
Loading status checks…
9374b9b
@bors-servo
Copy link
Contributor

bors-servo commented Jul 25, 2015

☔ The latest upstream changes (presumably #6660) made this pull request unmergeable. Please resolve the merge conflicts.
@dzbarsky dzbarsky closed this Jul 29, 2015
@dzbarsky dzbarsky deleted the dzbarsky:rangeobserver branch Aug 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
S-awaiting-review
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@dzbarsky @bors-servo @highfive
You can’t perform that action at this time.