Summary: | [Qt] tst_qwebpage does not pass | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Robert Hogan <robert> | ||||||
Component: | Tools / Tests | Assignee: | QtWebKit Unassigned <webkit-qt-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | jwieczorek | ||||||
Priority: | P2 | Keywords: | Qt | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Other | ||||||||
OS: | OS X 10.5 | ||||||||
Attachments: |
|
Description
Robert Hogan
2010-03-15 14:12:07 PDT
Created attachment 50737 [details]
Patch
Fixing unstable tests is always a good thing but usually it is better to fix the reason of a failure, I think. Here is the interesting bit of the crash: Program received signal SIGSEGV, Segmentation fault. 0x00007ffff7022dc2 in QWebView::setPage (this=0x71f3a0, page=0x0) at WebKit/qt/Api/qwebview.cpp:401 401 d->page->d->page->settings()->setAcceleratedCompositingEnabled(false); (gdb) bt full #0 0x00007ffff7022dc2 in QWebView::setPage (this=0x71f3a0, page=0x0) at WebKit/qt/Api/qwebview.cpp:401 Created attachment 50746 [details]
Patch
Actually fix the bug per Jakub's kind suggestion!
Comment on attachment 50746 [details]
Patch
Why the indent change?
Manually committed: http://trac.webkit.org/changeset/56060 (In reply to comment #4) > (From update of attachment 50746 [details]) > Why the indent change? It's moved into an if block. |