Bug 120758

Summary: [Qt][WK1] REGRESSION(r155086): page cache related tests crashing
Product: WebKit Reporter: Zoltan Arvai <zarvai>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Critical CC: abrhm, allan.jensen, andersca, kadam, kling, ossy
Priority: P1    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 79668, 120710    
Attachments:
Description Flags
backtrace none

Description Zoltan Arvai 2013-09-05 00:32:48 PDT
Page cache related tests crashing on Qt WK1 after the patch.

http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Debug/r155091%20%2830355%29/results.html

+http/tests/navigation/https-in-page-cache.html
+compositing/iframes/page-cache-layer-tree.html
+animations/resume-after-page-cache.html
+fast/media/media-query-list-04.html
+fast/viewport/viewport-128.html
+fast/workers/termination-early.html
+fast/forms/autocomplete-off-with-default-value-does-not-clear.html
+fast/loader/input-element-page-cache-crash.html
+fast/loader/stateobjects/popstate-fires-with-page-cache.html
+fast/loader/scroll-position-restored-on-back.html
+fast/loader/window-properties-restored-from-page-cache.html
+fast/images/animated-gif-restored-from-bfcache.html
+fast/harness/page-cache-crash-on-data-urls.html
+fast/harness/use-page-cache.html
+fast/frames/frame-crash-with-page-cache.html
+fast/overflow/horizontal-scroll-after-back.html
+fast/events/pageshow-pagehide-on-back-cached.html
+fast/events/pagehide-timeout.html
+fast/events/suspend-timers.html
+fast/events/pageshow-pagehide-on-back-cached-with-frames.html
+fast/events/pagehide-xhr-open.html
+loader/go-back-cached-main-resource.html

https-in-page-cache-crash-log.txt
crash log for DumpRenderTree (pid 30815):
STDOUT: ALERT: This page is https and has the no-store cache-control directive. It should NOT go in to the page cache.
STDOUT: ALERT: The page was reloaded on back, not from the page cache. Good job. Running part 2 of the test.
STDOUT: ALERT: This page is https and has the no-cache cache-control directive. It should NOT go in to the page cache.
STDOUT: ALERT: The page was reloaded on back, not from the page cache. Good job. Running part 3 of the test.
STDOUT: ALERT: This page is https and should go in to the page cache.
STDERR: ASSERTION FAILED: m_ptr
STDERR: /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/Source/WTF/wtf/OwnPtr.h(72) : WTF::OwnPtr<T>::ValueType* WTF::OwnPtr<T>::operator->() const [with T = WebCore::CachedFrame, WTF::OwnPtr<T>::PtrType = WebCore::CachedFrame*, WTF::OwnPtr<T>::ValueType = WebCore::CachedFrame]
STDERR: 1   0x7f24326be49b /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/WebKitBuild/Debug/lib/libQt5WebKit.so.5(WTFCrash+0x1e) [0x7f24326be49b]
STDERR: 2   0x7f243132a3e8 /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/WebKitBuild/Debug/lib/libQt5WebKit.so.5(+0xa523e8) [0x7f243132a3e8]
....
Comment 1 Zoltan Arvai 2013-09-05 01:32:22 PDT
Created attachment 210589 [details]
backtrace

Backtrace for fast/loader/input-element-page-cache-crash.html.
Comment 2 Andreas Kling 2013-09-05 08:17:07 PDT
Committed r155120: <http://trac.webkit.org/changeset/155120>