Bug 172845
| Summary: | DOMTokenList.replace('a', 'a') removes 'a'. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Kent Tamura <tkent> |
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WORKSFORME | ||
| Severity: | Major | CC: | cdumez |
| Priority: | P2 | ||
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Kent Tamura
1. Open http://w3c-test.org/dom/nodes/Element-classlist.html
2. Observe "classList.replace("a", "a") with attribute value "a a a b" (HTML node)" test.
This test fails with the following message:
> assert_equals: wrong class after modification expected "a b" but got "b"
Existing "a" token was unexpectedly removed.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Chris Dumez
Pass classList.replace("a", "a") with attribute value "a a a b" (HTML node)
So presumably this has already been fixed?
Chris Dumez
(In reply to Chris Dumez from comment #1)
> Pass classList.replace("a", "a") with attribute value "a a a b" (HTML node)
>
> So presumably this has already been fixed?
Likely a dupe of Bug 171388. I am confirming.
Chris Dumez
(In reply to Chris Dumez from comment #2)
> (In reply to Chris Dumez from comment #1)
> > Pass classList.replace("a", "a") with attribute value "a a a b" (HTML node)
> >
> > So presumably this has already been fixed?
>
> Likely a dupe of Bug 171388. I am confirming.
I can reproduce the bug with shipping Safari but not with Safari Technology 30 (which is not even the latest one).
Kent Tamura
Oh, I tested with Safari TP 29. Sorry for the noise.
Chris Dumez
(In reply to Kent Tamura from comment #4)
> Oh, I tested with Safari TP 29. Sorry for the noise.
No worries. Thank you for reporting such bugs.