Moved setting of visual ordering from DocumentWriter to DecodedDataDocument

Source/core/dom/DocumentParser.h

Index: Source/core/dom/DocumentParser.h
diff --git a/Source/core/dom/DocumentParser.h b/Source/core/dom/DocumentParser.h
index 1b0e91f8a52579d5f2d2cb2d7c393b79334941eb..fa1d90a40c3fc2bf359f40bed0799789d4b054b2 100644
--- a/Source/core/dom/DocumentParser.h
+++ b/Source/core/dom/DocumentParser.h
@@ -49,11 +49,12 @@ public:
     virtual void insert(const SegmentedString&) = 0;
 
     // The below functions are used by DocumentWriter (the loader).
-    virtual size_t appendBytes(const char* bytes, size_t length) = 0;
-    virtual size_t flush() = 0;
+    virtual void appendBytes(const char* bytes, size_t length) = 0;
+    virtual void flush() = 0;
     virtual bool needsDecoder() const { return false; }
     virtual void setDecoder(PassRefPtr<TextResourceDecoder>);
     virtual PassRefPtr<TextResourceDecoder> decoder();
+    virtual void setHasAppendedData() { }

[eseidel, 2013/11/18 23:46:08]

Curious why this needs to be defined here?

[oystein (OOO til 10th of July), 2013/11/18 23:51:56]

Even more strange hackiness :). DocumentWriter::appendReplacingData calls
DocumentParser::append directly with already decoded bytes rather than sending
raw bytes to DocumentParser::appendBytes. And for some reason that means it also
has to block the setting of the "visually ordered" thing which would happen
first time appendBytes() is called. It sounds weird to me, but that was how the
existing behavior was :/.

 
     // pinToMainThread also makes append() not yield before completion of that chunk.
     virtual void pinToMainThread() { }