2010-07-23 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Jul 2010 00:03:53 +0000 (00:03 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Jul 2010 00:03:53 +0000 (00:03 +0000)
        Remove FIXME that we already fixed.

        * html/HTMLTreeBuilder.cpp:
        (WebCore::HTMLTreeBuilder::processCharacter):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@64001 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/html/HTMLTreeBuilder.cpp

index 0b1f5ee..4e2bf74 100644 (file)
@@ -1,3 +1,10 @@
+2010-07-23  Adam Barth  <abarth@webkit.org>
+
+        Remove FIXME that we already fixed.
+
+        * html/HTMLTreeBuilder.cpp:
+        (WebCore::HTMLTreeBuilder::processCharacter):
+
 2010-06-24  Dimitri Glazkov  <dglazkov@chromium.org>
 
         Reviewed by Kent Tamura.
index 6aca72f..4f6ce02 100644 (file)
@@ -2439,10 +2439,6 @@ void HTMLTreeBuilder::processComment(AtomicHTMLToken& token)
 void HTMLTreeBuilder::processCharacter(AtomicHTMLToken& token)
 {
     ASSERT(token.type() == HTMLToken::Character);
-    // FIXME: Currently this design has an extra memcpy because we copy the
-    // characters out of the HTMLTokenizer's buffer into the AtomicHTMLToken
-    // and then into the text node.  What we'd really like is to copy directly
-    // from the HTMLTokenizer's buffer into the text node.
     ExternalCharacterTokenBuffer buffer(token);
     processCharacterBuffer(buffer);
 }