Add AtomicHTMLToken
Created attachment 59614 [details] Patch
Comment on attachment 59614 [details] Patch I like the idea. It should have its own header though. It could be a subclass of HTMLToken, but I think it's nice to be separate like this. HTMLToken may end up being a struct and internal to the Tokenizer as you suggest. I'm ready to r+ this, but I think it needs its own header.
> It could be a subclass of HTMLToken, but I think it's nice to be separate like this. HTMLToken may end up being a struct and internal to the Tokenizer as you suggest. It can't be a subclass because we don't want the large buffers that HTMLToken has. > I'm ready to r+ this, but I think it needs its own header. I could add a separate header, but it's eventually going to eat HTMLToken, so there doesn't seem to be much of a point in adding a new header.
(In reply to comment #3) > > It could be a subclass of HTMLToken, but I think it's nice to be separate like this. HTMLToken may end up being a struct and internal to the Tokenizer as you suggest. > > It can't be a subclass because we don't want the large buffers that HTMLToken has. Agreed. > > I'm ready to r+ this, but I think it needs its own header. > > I could add a separate header, but it's eventually going to eat HTMLToken, so there doesn't seem to be much of a point in adding a new header. I don't believe you.
Comment on attachment 59614 [details] Patch I think your lack-of-new-header is mostly being lazy. :p
Committed r61791: <http://trac.webkit.org/changeset/61791>