[webkit-reviews] review granted: [Bug 13942] ASSERTION FAILED: !attrName.contains('/') in HTMLTokenizer.cpp:132 when loading http://bamanzi.blogeden.cn/ : [Attachment 20706] Check for '/' when assigning value to attribute to avoid triggering the assertion

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Julien Chaffraix
<julien.chaffraix at gmail.com>'s request for review:
Bug 13942: ASSERTION FAILED: !attrName.contains('/') in HTMLTokenizer.cpp:132
when loading http://bamanzi.blogeden.cn/
http://bugs.webkit.org/show_bug.cgi?id=13942

Attachment 20706: Check for '/' when assigning value to attribute to avoid
triggering the assertion
http://bugs.webkit.org/attachment.cgi?id=20706&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
Is the "/" character the only bad character for an attribute name?

This change seems fine as far as it goes, but I'm surprised that this is the
only character that is allowed in attribute values but can cause us trouble in
attribute names. Maybe ":"?

r=me, but lets consider further testing with other characters