[webkit-reviews] review denied: [Bug 49468] [Qt] Assertion failure in DocumentLoader::commitData when loading a media document : [Attachment 91036] Cancel load of top level media content before reset/assert

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 26 16:45:23 PDT 2011


Simon Hausmann <hausmann at webkit.org> has denied Joe Wild
<joseph.wild at nokia.com>'s request for review:
Bug 49468: [Qt] Assertion failure in DocumentLoader::commitData when loading a
media document
https://bugs.webkit.org/show_bug.cgi?id=49468

Attachment 91036: Cancel load of top level media content before reset/assert
https://bugs.webkit.org/attachment.cgi?id=91036&action=review

------- Additional Comments from Simon Hausmann <hausmann at webkit.org>
I don't think that this is the correct fix. The assertion is correct I believe,
and this works in the other ports. The cancellation of the first load should be
issued by FrameLoaderClientQt::committedLoad.

See also http://webkit.org/b/48762 and http://trac.webkit.org/changeset/51104


More information about the webkit-reviews mailing list