hasAttribute always returns false for uppercase attribute names like "STYLE"

[Darin Adler]

This is especially important because of a major problem it causes in the Mail application.

[Darin Adler]

Created attachment 7061 [details]
patch with detailed change log and a layout test

[Darin Adler]

<rdar://problem/4474884> hasAttribute always returns false for uppercase attribute names like "STYLE"

[Geoffrey Garen]

Comment on attachment 7061 [details]
patch with detailed change log and a layout test

The new layout test "console" is neat.

"Did not raise an exception as expected" sounds ambiguous to me. I would say, "was expected to raise an exception but didn't" instead.

Am I right that lowercasedName is only actually lower case in the HTML case? Maybe we should just call it something like nameCopy instead.

r=me

[Eric Seidel (no email)]

Comment on attachment 7061 [details]
patch with detailed change log and a layout test

I find calling your local variable "lowercasedNamed" sorta odd, when half the time it's actually not lowercased.

You might consider calling it localName or name depending on which of those is already taken.

[Eric Seidel (no email)]

Oops, just call me "ggaren2".

[Lucas Forschler]

Mass moving XML DOM bugs to the "DOM" Component.