26439 2009-06-16 03:04:37 -0700 [Qt] QtWebKit fails in loading www.nytimes.com in Windows/Linux 2009-07-10 06:29:17 -0700 1 1 1 Unclassified WebKit Page Loading 528+ (Nightly build) PC All RESOLVED FIXED http://www.nytimes.com P2 Normal --- 1 tiansjun sriram.yadavalli hausmann mjs sriram.yadavalli tonikitoo oldest_to_newest 126179 0 tiansjun 2009-06-16 03:04:37 -0700 While loading www.nytimes.com, the top most bar cannot be loaded correctly. Here is the corrupt image: http://lh6.ggpht.com/_7B4pgaO4oRQ/ShlwpU8LuKI/AAAAAAAAALw/WnRC34KnjDY/s912/nytimes.png Here is the correct one: http://lh6.ggpht.com/_7B4pgaO4oRQ/SjdspfyF5II/AAAAAAAAAMM/hLTZ37gGA4g/s912/correct.png I have test Qtlauncher,arora in both Windows/Linux and Konqueror. All fail to load nytimes correctly. The bug only occurs in Qt port. I have test QtWebKit version 4.3.3, version 4.5.1 and the latest nightly webkit. Anyone works on Qt port of webkit can solve this problem? Thanks a lot. 129451 1 tiansjun 2009-07-01 21:26:35 -0700 May it be that no loadFinished signal emit for javascript created content? 129797 2 32256 sriram.yadavalli 2009-07-04 11:06:58 -0700 Created attachment 32256 Fixes issue associated with rejecting response of HTTP 401 error In Qt 4.5, the QNetworkReply doesnt call finish for a 401. In Qt4.6, QNetworkReply signals for 401 works, but QNetworkReplyHandler was not handling it properly. 129851 3 tiansjun 2009-07-05 04:39:53 -0700 It seems not to be fixed yet. I update this patch into webkit-r44683, but Qtlauncher still hangs on 90%... 129857 4 sriram.yadavalli 2009-07-05 07:00:19 -0700 This issue needs fix in Qt layer and in Qt Webkit. Please get the latest Qt4.6 from gitorious.org and the latest Webkit (with this patch) to see the patch work. 129861 5 tiansjun 2009-07-05 08:06:26 -0700 (In reply to comment #4) > This issue needs fix in Qt layer and in Qt Webkit. Please get the latest Qt4.6 > from gitorious.org and the latest Webkit (with this patch) to see the patch > work. Okay,I will try it. Thanks for your work! 130873 6 32256 hausmann 2009-07-10 06:28:19 -0700 Comment on attachment 32256 Fixes issue associated with rejecting response of HTTP 401 error r=me. thanks, good catch! Two minor points though: 1) Your patch doesn't apply as-is because the diff has relative paths in it (../..) 2) Please don't put in my name as reviewer before it has been reviewed. 130874 7 hausmann 2009-07-10 06:29:17 -0700 Landed in r45707. I've also added ProxyAuthenticationError to the same condition, after a quick discussion with Prasanth. 32256 2009-07-04 11:06:58 -0700 2009-07-10 06:28:18 -0700 Fixes issue associated with rejecting response of HTTP 401 error BugFix401.patch text/plain 1877 sriram.yadavalli SW5kZXg6IC4uLy4uL1dlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIC4uLy4uL1dlYkNv cmUvQ2hhbmdlTG9nCShyZXZpc2lvbiA0NTU0NikKKysrIC4uLy4uL1dlYkNvcmUvQ2hhbmdlTG9n CSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMDktMDctMDQgIFNyaXJhbSBZYWRh dmFsbGkgIDxzcmlyYW0ueWFkYXZhbGxpQG5va2lhLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBi eSBTaW1vbiBIYXVzbWFubiAoaGF1c21hbm5Ad2Via2l0Lm9yZykuCisKKyAgICAgICAgUXRXZWJL aXQgZmFpbHMgaW4gbG9hZGluZyB3d3cubnl0aW1lcy5jb20gaW4gV2luZG93cy9MaW51eAorICAg ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjY0MzkKKyAgICAg ICAgUU5ldHdvcmtSZXBseUhhbmRsZXIgaXMgaWdub3JpbmcgY29udGVudCBhc3NvY2lhdGVkIHdp dGggNDAxIGVycm9yLiAgVGhpcyBjYXVzZXMgdGhlIFhIUiByZXNwb25zZSBoYW5kbGluZyB0byBm YWlsLgorCisgICAgICAgICogcGxhdGZvcm0vbmV0d29yay9xdC9RTmV0d29ya1JlcGx5SGFuZGxl ci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpRTmV0d29ya1JlcGx5SGFuZGxlcjo6ZmluaXNoKToK KwogMjAwOS0wNy0wMyAgRGFyaW4gQWRsZXIgIDxkYXJpbkBhcHBsZS5jb20+CiAKICAgICAgICAg UmV2aWV3ZWQgYnkgQWRlbGUgUGV0ZXJzb24uCkluZGV4OiAuLi8uLi9XZWJDb3JlL3BsYXRmb3Jt L25ldHdvcmsvcXQvUU5ldHdvcmtSZXBseUhhbmRsZXIuY3BwCj09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIC4uLy4u L1dlYkNvcmUvcGxhdGZvcm0vbmV0d29yay9xdC9RTmV0d29ya1JlcGx5SGFuZGxlci5jcHAJKHJl dmlzaW9uIDQ1NTQ1KQorKysgLi4vLi4vV2ViQ29yZS9wbGF0Zm9ybS9uZXR3b3JrL3F0L1FOZXR3 b3JrUmVwbHlIYW5kbGVyLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMjI1LDkgKzIyNSwxMCBAQAog ICAgICAgICByZXNldFN0YXRlKCk7CiAgICAgICAgIHN0YXJ0KCk7CiAgICAgfSBlbHNlIGlmICht X3JlcGx5LT5lcnJvcigpICE9IFFOZXR3b3JrUmVwbHk6Ok5vRXJyb3IKLSAgICAgICAgICAgICAg IC8vIGEgd2ViIHBhZ2UgdGhhdCByZXR1cm5zIDQwMy80MDQgY2FuIHN0aWxsIGhhdmUgY29udGVu dAorICAgICAgICAgICAgICAgLy8gYSB3ZWIgcGFnZSB0aGF0IHJldHVybnMgNDAxLzQwMy80MDQg Y2FuIHN0aWxsIGhhdmUgY29udGVudAogICAgICAgICAgICAgICAgJiYgbV9yZXBseS0+ZXJyb3Io KSAhPSBRTmV0d29ya1JlcGx5OjpDb250ZW50T3BlcmF0aW9uTm90UGVybWl0dGVkRXJyb3IKLSAg ICAgICAgICAgICAgICYmIG1fcmVwbHktPmVycm9yKCkgIT0gUU5ldHdvcmtSZXBseTo6Q29udGVu dE5vdEZvdW5kRXJyb3IpIHsKKyAgICAgICAgICAgICAgICYmIG1fcmVwbHktPmVycm9yKCkgIT0g UU5ldHdvcmtSZXBseTo6Q29udGVudE5vdEZvdW5kRXJyb3IKKyAgICAgICAgICAgICAgICYmIG1f cmVwbHktPmVycm9yKCkgIT0gUU5ldHdvcmtSZXBseTo6QXV0aGVudGljYXRpb25SZXF1aXJlZEVy cm9yKSB7CiAgICAgICAgIFFVcmwgdXJsID0gbV9yZXBseS0+dXJsKCk7CiAgICAgICAgIFJlc291 cmNlRXJyb3IgZXJyb3IodXJsLmhvc3QoKSwgbV9yZXBseS0+YXR0cmlidXRlKFFOZXR3b3JrUmVx dWVzdDo6SHR0cFN0YXR1c0NvZGVBdHRyaWJ1dGUpLnRvSW50KCksCiAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgdXJsLnRvU3RyaW5nKCksIG1fcmVwbHktPmVycm9yU3RyaW5nKCkpOwo=