[Webkit-unassigned] [Bug 16179] any attribute name start with a unicode which like #xx00(x could be any hex number[0-9a-f]) will cause HTMLTokenizer parse error.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 5 16:25:40 PST 2007


darin at apple.com changed:

           What    |Removed                     |Added
  Attachment #17727|review?                     |review-
               Flag|                            |

------- Comment #9 from darin at apple.com  2007-12-05 16:25 PDT -------
(From update of attachment 17727)
+                    // from statck, its' length is 10, should be OK.

Misspellings here: statck should be stack and its' should be its.

+                    unsigned int testedEntityNameLen = 0;

We normally delcar these as unsigned.

+                        if (cBuffer[testedEntityNameLen] > 0xff)

Seems to me we could use a smaller cutoff. Only ASCII characters can be in the
entity names, so the cutoff could be 7F or 7E.

+                            break;
+                        else
+                            chTmpEntityNameBuffer[testedEntityNameLen] =

No need for an else after a break; we don't like to nest code like this.

The layout test should not be in the webarchive directory -- it's not a test of
web archives. It should go into the fast/parser directory since it's a test of
the HTML parser.

Also, the test should use "dumpAsText()" so it can work cross-platform.

Since this is super-hot code, we need to do some performance testing to make
sure it doesn't slow things down.

Configure bugmail: http://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.

More information about the webkit-unassigned mailing list