Bug 93628 - HTMLElement.classList cannot remove classnames with uppercase characters
Summary: HTMLElement.classList cannot remove classnames with uppercase characters
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Erik Arvidsson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2012-08-09 10:22 PDT by Alexey Proskuryakov
Modified: 2012-11-08 12:44 PST (History)
2 users (show)

See Also:

Attachments
test case (313 bytes, text/html)
2012-08-09 10:22 PDT, Alexey Proskuryakov 		no flags Details
test case (298 bytes, text/html)
2012-08-09 10:46 PDT, Alexey Proskuryakov 		no flags Details
Patch (6.16 KB, patch)
2012-08-09 17:09 PDT, Erik Arvidsson 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2012-08-09 10:22:59 PDT 
Created attachment 157480 [details]
test case

In certain circumstances, HTMLElement.classList has trouble with case sensitivity.

See attached test case. What's interesting is that adding a class attribute to the <body> element in markup makes this test pass.

<rdar://problem/12034524>
Comment 1 Erik Arvidsson 2012-08-09 10:30:06 PDT 
This is quirks mode but it should still work.
Comment 2 Alexey Proskuryakov 2012-08-09 10:46:21 PDT 
Created attachment 157488 [details]
test case

Yeah, re-uploading the test case without an incorrect doctype to avoid future confusion.
Comment 3 Erik Arvidsson 2012-08-09 15:00:05 PDT 
One more data point. This does not happen on a disconnected DIV element.
Comment 4 Erik Arvidsson 2012-08-09 17:09:50 PDT 
Created attachment 157588 [details]
Patch
Comment 5 WebKit Review Bot 2012-08-09 19:26:51 PDT 
Comment on attachment 157588 [details]
Patch

Clearing flags on attachment: 157588

Committed r125239: <http://trac.webkit.org/changeset/125239>
Comment 6 WebKit Review Bot 2012-08-09 19:26:54 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 7 Alexey Proskuryakov 2012-11-08 12:44:42 PST 
This has caused a regression, bug 101627.