Summary: | REGRESSION (r137286-r137296): unit test failure in WebKit2APITests/TestWebKitWebContext | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Zan Dobersek <zan> | ||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cgarcia, gustavo, mrobinson, rego, spenap, webkit.review.bot | ||||
Priority: | P2 | Keywords: | Gtk, Regression | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 113282, 113707 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Zan Dobersek
2012-12-12 01:44:23 PST
Skipped in r137434. http://trac.webkit.org/changeset/137434 Was treating this as a regression after r137198, but that patch has since been rolled out and the test is still failing. http://trac.webkit.org/changeset/137198 Perhaps this is related to the recent MainResourceLoader modifications, for instance r137333 and similar patches. http://trac.webkit.org/changeset/137333 Good thing I have time for a bisection! Tested this with r144170 and the failure still occurs. *** Bug 112421 has been marked as a duplicate of this bug. *** I've been taking a look to this issue, I don't have too much information but I'd like to share it here. On one hand, the test works for me in the stable branch, however it's failing in trunk (and it was already failing when I reported the duplicated bug #112421). On the other hand, it seems that the problem is that DocumentLoader::mainResourceData() is returning NULL. This means that dataReference in WebPage::getMainResourceDataOfFrame() is not properly initialized. It doesn't fail for me with a recent trunk (r146868) (In reply to comment #6) > It doesn't fail for me with a recent trunk (r146868) It's working for me too in r147306. Could someone unskip it and close this, please? Created attachment 195947 [details]
Patch
Comment on attachment 195947 [details] Patch Clearing flags on attachment: 195947 Committed r147319: <http://trac.webkit.org/changeset/147319> All reviewed patches have been landed. Closing bug. Re-opened since this is blocked by bug 113707 (In reply to comment #12) > Re-opened since this is blocked by bug 113707 The test is still failing on the 64-bit release builder, so rolling the unskip patch out. The failure is still the same. (In reply to comment #13) > (In reply to comment #12) > > Re-opened since this is blocked by bug 113707 > > The test is still failing on the 64-bit release builder, so rolling the unskip patch out. The failure is still the same. hmm, I managed to reproduce the issue in the 64 bit release bot. It seems the scheme handlers are registered correctly, but libsoup is not handling them. I've just built a new libsoup in the bot and it works, so this will be fixed when bug #113282 lands. Comment on attachment 195947 [details] Patch Bug #113282 landed, perhaps just flipping the cq flag on this patch will do all the magic of landing, removing the need to reupload the same patch. Comment on attachment 195947 [details]
Patch
Wait a bit bots still don't build, and the patch has Reviewed by NOBODY
Ugh, right. Sorry for jumping forward with it. Comment on attachment 195947 [details]
Patch
Sorry, it seems the 64 bit release bot is building fine now.
Comment on attachment 195947 [details] Patch Clearing flags on attachment: 195947 Committed r147559: <http://trac.webkit.org/changeset/147559> All reviewed patches have been landed. Closing bug. |