RESOLVED FIXED 41623
Implement basic text node coalescing 
https://bugs.webkit.org/show_bug.cgi?id=41623
Summary Implement basic text node coalescing
Adam Barth
Reported 2010-07-05 10:01:38 PDT
Implement basic text node coalessing
Attachments
Patch (13.48 KB, patch)
2010-07-05 10:05 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Patch for landing (14.35 KB, patch)
2010-07-05 21:11 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2010-07-05 10:05:46 PDT
Created attachment 60551 [details] Patch
Eric Seidel (no email)
Comment 2 2010-07-05 21:04:44 PDT
Comment on attachment 60551 [details] Patch WebCore/dom/CharacterData.cpp:62 + PassRefPtr<StringImpl> CharacterData::parserAppendData(const String& arg) We need a bunch more of these parser* functions. WebCore/dom/CharacterData.h:45 + // Similar to appendData, but optimized for the parser (e.g., no mutation I would have used "Like appendData" to keep it to one 80c line. :) WebCore/html/HTMLTreeBuilder.cpp:1583 + // was the last text node inserted by the parser. Can we write a test which fails this so we know when we fix it? Or do we know it's already covered?
Adam Barth
Comment 3 2010-07-05 21:11:23 PDT
Created attachment 60579 [details] Patch for landing
WebKit Commit Bot
Comment 4 2010-07-05 22:15:03 PDT
Comment on attachment 60579 [details] Patch for landing Clearing flags on attachment: 60579 Committed r62520: <http://trac.webkit.org/changeset/62520>
WebKit Commit Bot
Comment 5 2010-07-05 22:15:08 PDT
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 6 2010-07-06 08:26:44 PDT
http://trac.webkit.org/changeset/62520 might have broken Leopard Intel Release (Tests)
Note You need to log in before you can comment on or make changes to this bug.