Summary: | [Qt] REGRESSION(r58497) tst_QGraphicsWebView::crashOnViewlessWebPages() is failing | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jesus Sanchez-Palencia <jesus> | ||||||
Component: | Tools / Tests | Assignee: | Kenneth Rohde Christiansen <kenneth> | ||||||
Status: | CLOSED FIXED | ||||||||
Severity: | Normal | CC: | benjamin, commit-queue, eric, hausmann, kenneth, kling, tonikitoo | ||||||
Priority: | P2 | Keywords: | Qt | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 38654 | ||||||||
Attachments: |
|
Description
Jesus Sanchez-Palencia
2010-05-06 10:19:05 PDT
reproducible on both release and debud builds. I can have a look this autotest has a failing history: * r56529 - NO BUG - broke it * r57323 - bug 37317 - fixed it * r58497 - bug 38179 - broke it again The bug is causes by the fact that a resize of an empty page causes a layout, thus deleting the qgraphicswebview before setHtml is called, which then deletes it again, causing a double free. Patch coming. Created attachment 55719 [details]
Patch
Created attachment 55730 [details]
Patch 2 (fixes typo)
Comment on attachment 55730 [details]
Patch 2 (fixes typo)
Looks good to me, r+.
Comment on attachment 55730 [details] Patch 2 (fixes typo) Rejecting patch 55730 from commit-queue. Unexpected failure when landing patch! Please file a bug against webkit-patch. Failed to run "['../WebKit/WebKitTools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', 'land-attachment', '--force-clean', '--build', '--non-interactive', '--ignore-builders', '--build-style=both', '--quiet', 55730, '--test', '--parent-command=commit-queue', '--no-update']" exit_code: 1 Last 500 characters of output: id=55730&action=edit Fetching: https://bugs.webkit.org/show_bug.cgi?id=38655&ctype=xml Processing 1 patch from 1 bug. Cleaning working directory Processing patch 55730 from bug 38655. NOBODY (OOPS!) found in /Users/eseidel/Projects/CommitQueue/WebKit/qt/ChangeLog does not appear to be a valid reviewer according to committers.py. ERROR: /Users/eseidel/Projects/CommitQueue/WebKit/qt/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive). This false rejection is caused by bug 38998. This won't work in the commit-queue until that's fixed, sorry. :( Committed r59378: <http://trac.webkit.org/changeset/59378> Revision r59378 cherry-picked into qtwebkit-2.0 with commit 3e4eb12fb842fa1709551e05c52fc695014a21fa |