Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-06 06:00:15 +01:00
Code Issues Projects Releases Packages Wiki Activity

Remove obsolete step from Text::SplitText

Browse source 
https://github.com/whatwg/dom/pull/419
This commit is contained in:
Anthony Ramine 2017-03-07 13:31:22 +01:00
parent b66839367c
commit 57291e5609
2 changed files with 3 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

6
components/script/dom/range.rs
View file

@ -1214,12 +1214,6 @@ impl WeakRangeVec {
}
}
/// Used for steps 9.1-2. when splitting a text node.
/// https://dom.spec.whatwg.org/#concept-text-split
pub fn clamp_above(&self, node: &Node, offset: u32) {
self.map_offset_above(node, offset, |_| offset);
}
fn map_offset_above<F: FnMut(u32) -> u32>(&self, node: &Node, offset: u32, mut f: F) {
unsafe {
(*self.cell.get()).update(|entry| {

9
components/script/dom/text.rs
View file

@ -43,7 +43,8 @@ impl Text {
}
impl TextMethods for Text {
// https://dom.spec.whatwg.org/#dom-text-splittextoffset
// https://dom.spec.whatwg.org/#dom-text-splittext
// https://dom.spec.whatwg.org/#concept-text-split
fn SplitText(&self, offset: u32) -> Fallible<Root<Text>> {
let cdata = self.upcast::<CharacterData>();
// Step 1.
@ -72,11 +73,7 @@ impl TextMethods for Text {
}
// Step 8.
cdata.DeleteData(offset, count).unwrap();
if parent.is_none() {
// Step 9.
node.ranges().clamp_above(&node, offset);
}
// Step 10.
// Step 9.
Ok(new_node)
}