[Webkit-unassigned] [Bug 21076] New: incorrect tabindex parsing

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=21076

           Summary: incorrect tabindex parsing
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: alexmac at adobe.com


When an unquoted attribute appears atthe end of an element with no space
between it and the closing "/>" the attribute will have a trailing "/" appended
to its value, this is a bug in the tokenizer, although it seems to be kept to
maintain compatibility with old versions of IE

however when a tabindex attribute is given this extra trailing slash it fails
the new tabindex HTML5 validation in HTMLElement::parseMappedAttribute

This behaviour does not occur on FF3.0.2 or IE7

Checking for, and removing, a single trailing slash fixes the problem. Here is
the code I'm using to fix the problem, its inserted after line 147 in
HTMLElement.cpp:

if(indexstring[indexstring.length()-1] == '/') {
  indexstring.truncate(indexstring.length()-1);
}

Does this seem reasonable, or is there a better way to fix it?


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.