66749 2011-08-23 00:27:39 -0700 [Qt] Assertion fails in three regression tests after r93567 2012-05-17 16:10:31 -0700 1 1 1 Unclassified WebKit Frames 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 53191 79666 79668 1 rhodovan.u-szeged rafael.lobo allan.jensen ap ggaren lakshmiprasannag10 noam ossy rafael.lobo zherczeg oldest_to_newest 455225 0 rhodovan.u-szeged 2011-08-23 00:27:39 -0700 Three tests have assertion error in debug mode on x86-32 Linux Qt: ASSERTION FAILED: webFrame.isNull() ../../../../Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp(1323) : virtual WTF::PassRefPtr<WebCore::Frame> WebCore::FrameLoaderClientQt::createFrame(const WebCore::KURL&, const WTF::String&, WebCore::HTMLFrameOwnerElement*, const WTF::String&, bool, int, int) Segmentation fault The failed tests are: fast/frames/iframe-onload-remove-self-no-crash.html fast/loader/create-frame-in-DOMContentLoaded.html fast/loader/frame-creation-removal.html 455244 1 rhodovan.u-szeged 2011-08-23 01:25:48 -0700 Committed r93581: <http://trac.webkit.org/changeset/93581> 455246 2 rhodovan.u-szeged 2011-08-23 01:28:12 -0700 The error is still exist. 455457 3 ap 2011-08-23 09:56:36 -0700 Why was this assigned to me? Even provided that it started with my check-in, I have no way to work on this bug. Notably, Mac WebKit1 also has this assertion, which doesn't get hit. 470627 4 lakshmiprasannag10 2011-09-21 00:23:14 -0700 The Asset condition is checking webFrame object existence, it has not been freed any where before the Assert instruction, so the webFrame.isNull() instruction is returning false. After adding following instructions all the test cases mentioned above are passing. delete webFrame ; ASSERT(webFrame.isNull()); 617454 5 allan.jensen 2012-05-07 01:32:17 -0700 I think the assert can be removed. The frame is not always deleted right away, and none of the other implementations of FrameLoaderClient has a similar assert. 627044 6 rafael.lobo 2012-05-17 15:23:14 -0700 I think the first step is to figure out why there's this assumption that it will be deleted at that point. I will take a look. 627094 7 rafael.lobo 2012-05-17 16:10:31 -0700 It looks like this is fixed already. Tests are passing and I can't find these ones skipped. Closing bug.