[webkit-reviews] review requested: [Bug 7761] Tabs in class attribute not treated as whitespace : [Attachment 7089] Revised patch

[bugzilla-request-daemon at opendarwin.org]

opendarwin.org at mitzpettel.com has asked  for review:
Bug 7761: Tabs in class attribute not treated as whitespace
http://bugzilla.opendarwin.org/show_bug.cgi?id=7761

Attachment 7089: Revised patch
http://bugzilla.opendarwin.org/attachment.cgi?id=7089&action=edit

------- Additional Comments from opendarwin.org at mitzpettel.com
(In reply to comment #3)
> OK, let me now be crazy and make premature optimization comments:

Yay! I followed your suggestions in this version.

> Do we want to optimize the common case [...] so that we
> don't create a new atomic string for that case?

In that case, this patch uses AtomicString(classAttr) instead of
AtomicString(str, length), but I almost convinced myself that there is no
difference.

> Not sure what to do about these nitpicks. The code is good as is.

Not really... it was also checking each boundary character twice!