Add a test that we resume commits after inserting the body.

public/web/WebViewClient.h

Index: public/web/WebViewClient.h
diff --git a/public/web/WebViewClient.h b/public/web/WebViewClient.h
index f1ffa139f0b19a0dc9e1112fa9748786e324eea7..e5f1eae17f02eefcd0969cf402e50421bf161307 100644
--- a/public/web/WebViewClient.h
+++ b/public/web/WebViewClient.h
@@ -115,6 +115,10 @@ public:
     // A data url from <canvas> or <img> is passed to the method's argument.
     virtual void saveImageFromDataURL(const WebString&) { }
 
+    // Used by unit tests so that all parsing will be main thread for controling

[dcheng, 2015/08/27 23:10:22]

Nit: controlling

[esprehn, 2015/08/28 08:31:43]

Done.

+    // parsing and chunking precisely.
+    virtual bool disableThreadedParsingForUnitTests() const { return false; }

[dcheng, 2015/08/27 23:10:22]

This feels a bit out of place on WebViewClient. One thought was to dump this
into Internals.h (though with the exception of a few things, that appears to
largely be the implementation of Internals.idl...)

It would be even nicer if there was a way to be notified (in tests) when the
parser is has consumed all the input its received, rather than forcing threaded
parsing to be disabled. +kouhei who might know a way?

[esprehn, 2015/08/28 08:31:44]

Internals is for layout tests, you cannot access it from inside core. I'll make
this a static on document instead.

I'd prefer we didn't use the threaded parser at all, I want the tests to be
predictable and threaded parsing is not easily predictable in terms of chunking.

I don't think we gain anything from that in the tests either, it all funnels
down into the same HTMLConstructionSite code paths. Threaded parsing shouldn't
impact rendering tests.

+
 
     // Editing -------------------------------------------------------------