Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-07-23 07:13:52 +01:00
Code Issues Projects Releases Packages Wiki Activity

Past names expire when form owner is reset

Browse source
This commit is contained in:
Patrick Shaughnessy 2020-01-07 11:57:52 -05:00
parent 1d645f3741
commit 9ff6717992
2 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
components/script/dom/htmlformelement.rs
View file

@ -1081,6 +1081,13 @@ impl HTMLFormElement {
.iter() .iter()
.position(|c| &**c == control) .position(|c| &**c == control)
.map(|idx| controls.remove(idx)); .map(|idx| controls.remove(idx));
// https://html.spec.whatwg.org/multipage#forms.html#the-form-element:past-names-map-5
// "If an element listed in a form element's past names map
// changes form owner, then its entries must be removed
// from that map."
let mut past_names_map = self.past_names_map.borrow_mut();
past_names_map.retain(|_k, v| v.0 != control);
} }
} }

3
tests/wpt/metadata/html/semantics/forms/the-form-element/form-nameditem.html.ini
View file

@ -15,6 +15,3 @@
[Trying to set an expando that would shadow an already-existing named property] [Trying to set an expando that would shadow an already-existing named property]
expected: FAIL expected: FAIL
[Past names map should work correctly]
expected: FAIL