Moved text decoding to the parser thread

Source/core/html/parser/HTMLDocumentParser.h

Index: Source/core/html/parser/HTMLDocumentParser.h
diff --git a/Source/core/html/parser/HTMLDocumentParser.h b/Source/core/html/parser/HTMLDocumentParser.h
index b41beae929c1c532d848ad2f2096f35b4cfab52a..c881dd2ab483a06242687450ba327a0b2f949016 100644
--- a/Source/core/html/parser/HTMLDocumentParser.h
+++ b/Source/core/html/parser/HTMLDocumentParser.h
@@ -29,6 +29,7 @@
 #include "core/dom/ParserContentPolicy.h"
 #include "core/dom/ScriptableDocumentParser.h"
 #include "core/fetch/ResourceClient.h"
+#include "core/fetch/TextResourceDecoder.h"
 #include "core/html/parser/BackgroundHTMLInputStream.h"
 #include "core/html/parser/CompactHTMLToken.h"
 #include "core/html/parser/HTMLInputStream.h"
@@ -95,6 +96,10 @@ public:
         TokenPreloadScannerCheckpoint preloadScannerCheckpoint;
     };
     void didReceiveParsedChunkFromBackgroundParser(PassOwnPtr<ParsedChunk>);
+    void didReceiveEncodingDataFromBackgroundParser(const DocumentEncodingData&);
+
+    virtual void appendBytes(const char* bytes, size_t length) OVERRIDE;

[eseidel, 2013/11/19 22:27:45]

I thought you changed this to Vector<char>?  I guess not in this case?

[oystein (OOO til 10th of July), 2013/11/19 22:52:19]

I just changed the buffer we pass to the BackgroundHTMLParser::appendBytes from
a SharedBuffer to a Vector<char>, not the appendBytes() in the "main" parser
hierarchy. I wouldn't mind having a look at changing that, but I suspect it may
be a bigger operation which may make more sense in a separate patch?

It means we're still doing a copy of the incoming data which would ideally be
avoided, but we were under the old codepath as well.

+    virtual void flush() OVERRIDE;
 
 protected:
     virtual void insert(const SegmentedString&) OVERRIDE;