50746 2010-12-09 03:14:30 -0800 [Qt] GZip-Compressed content is not shown if end-of-file marker is missing 2011-06-20 12:15:13 -0700 1 1 1 Unclassified WebKit Page Loading 528+ (Nightly build) PC Linux RESOLVED DUPLICATE 58727 Qt, QtTriaged P2 Normal --- 0 thomas.thrainer webkit-unassigned benjamin hausmann laszlo.gombos markus peter.hartmann robert webkit.review.bot oldest_to_newest 319375 0 76036 thomas.thrainer 2010-12-09 03:14:30 -0800 Created attachment 76036 Ignore HttpError when the HTTP status code is 200 and at least some data was received Seen on the german eBay site, when logging into "Mein eBay". The Problem is that the gzip-stream sent by the server is correct, but does not have an end-of-file marker (zcat says "unexpected end of file"). Other browsers (Chrome, Firefox) do load the page without any error messages. I attach a patch which fixes this problem, and does show all available content when the server has sent a status code of 200. Potentially, incomplete pages could be shown with this patch. I don't think that's an issue tough, because otherwise the QWebPage's error page is shown instead of any data. 320016 1 thomas.thrainer 2010-12-10 01:24:53 -0800 I just realized that patching QtWebKit alone is probably not enough. In Qt, qhttpnetworkconnectionchannel.cpp (what a mouth full!), in the expand(...) method around line 585, QHttpNetworkReplyPrivate::gunzipBodyPartially(...) is called. The return value is compared to Z_STREAM_END if the received data was complete, which fails as the end-of-stream marker is missing. The decompressed data is _not_ appended to the received data in this case, instead an error is emited. This will cause the last piece of data to be lost. The right thing to do would probably be to append the data if it was successfully received (i.e. ret >= Z_OK), emit readyRead and dataReadProgress anyways, and only finally emit an error if dataComplete && ret != Z_STREAM_END. But I think somebody more knowledgeable than me should have a look at this and decide what's best to do. For now, QtWebKit does handle the eBay-site happily with the attached patch, even if some bytes are probably missing at the end. I created a Qt-bug for this issue too (http://bugreports.qt.nokia.com/browse/QTBUG-16022). 335162 2 webkit.review.bot 2011-01-16 23:43:22 -0800 Attachment 76036 did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files']" exit_code: 1 Total errors found: 0 in 0 files If any of these errors are false positives, please file a bug against check-webkit-style. 361071 3 76036 benjamin 2011-03-02 14:18:52 -0800 Comment on attachment 76036 Ignore HttpError when the HTTP status code is 200 and at least some data was received This is a bad idea. If a page has status 200, and the network is disconnected in the middle of the load, the frame will report loadFinished as if everything was find. And this patch is without test and changelog :) 361386 4 thomas.thrainer 2011-03-02 23:35:02 -0800 Actually, this patch was not really meant to be included, but rather as illustration for the problem and a (possible) solution for it. I don't think that I have enough knowledge about the WebKit codebase such that I can really propose a real solution for the problem... I guess I have misunderstood the patch submission and review process in this bugzilla tough :-). 361461 5 benjamin 2011-03-03 02:23:56 -0800 (In reply to comment #4) > I guess I have misunderstood the patch submission and review process in this bugzilla tough :-). Ok, no problem. You had set the r? flags, this request review for landing the patch in WebKit. 423842 6 robert 2011-06-20 12:15:13 -0700 *** This bug has been marked as a duplicate of bug 58727 *** 76036 2010-12-09 03:14:30 -0800 2011-03-02 14:18:52 -0800 Ignore HttpError when the HTTP status code is 200 and at least some data was received 0008_tiitoo_show_incomplete_pages.patch text/plain 509 thomas.thrainer LS0tIGEvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3b3JrL3F0L1FOZXR3b3JrUmVwbHlIYW5kbGVyLmNw cAkyMDEwLTEyLTAyIDA5OjI1OjU5Ljk5NDc0NDAwMSArMDEwMAorKysgYi9XZWJDb3JlL3BsYXRm b3JtL25ldHdvcmsvcXQvUU5ldHdvcmtSZXBseUhhbmRsZXIuY3BwCTIwMTAtMTItMDkgMTE6NTA6 MDkuMDM0NTMwOTk4ICswMTAwCkBAIC0yMjksNyArMjI5LDcgQEAKICAgICBpZiAoaHR0cFN0YXR1 c0NvZGUgPT0gNDAxIHx8IGh0dHBTdGF0dXNDb2RlID09IDQwNykKICAgICAgICAgcmV0dXJuIHRy dWU7CiAKLSAgICBpZiAocmVjZWl2ZWREYXRhICYmIChodHRwU3RhdHVzQ29kZSA+PSA0MDAgJiYg aHR0cFN0YXR1c0NvZGUgPCA2MDApKQorICAgIGlmIChyZWNlaXZlZERhdGEgJiYgKChodHRwU3Rh dHVzQ29kZSA+PSA0MDAgJiYgaHR0cFN0YXR1c0NvZGUgPCA2MDApIHx8IGh0dHBTdGF0dXNDb2Rl ID09IDIwMCkpCiAgICAgICAgIHJldHVybiB0cnVlOwogCiAgICAgcmV0dXJuIGZhbHNlOwo=