Bug 41710

Summary: [GTK] Crashes when going back with page cache in unknown circunstances
Product: WebKit Reporter: Gustavo Noronha (kov) <gustavo>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: xan.lopez
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
stack trace
none
stack trace for a different crash
none
One more null-check none

Description Gustavo Noronha (kov) 2010-07-06 12:10:30 PDT
After we enabled page cache for our port, we found a number of crashes, and unwanted behavior that were caused by our unique scrolling implementation (we try to be as close to what GTK+ expects, which not always maps well to WebCore's expectations, which is why we have quite a bit of platform-specific code). After fixing a number of these, one crash still appears in bug reports fairly frequently. I am attaching a trace.
Comment 1 Gustavo Noronha (kov) 2010-07-06 12:11:49 PDT
Created attachment 60651 [details]
stack trace
Comment 2 Gustavo Noronha (kov) 2010-07-06 12:12:59 PDT
Since we haven't been able to reproduce consistently reproduce this crash, and the traces make it fairly credible that document is NULL, I landed a null check as 62546.
Comment 3 Gustavo Noronha (kov) 2010-07-09 11:44:25 PDT
Created attachment 61068 [details]
stack trace for a different crash
Comment 4 Gustavo Noronha (kov) 2010-07-09 11:50:23 PDT
Created attachment 61070 [details]
One more null-check
Comment 5 Gustavo Noronha (kov) 2010-07-09 11:51:04 PDT
I'll reuse this bug, since it is tightly related to the other fix that points here.
Comment 6 Xan Lopez 2010-07-12 06:27:25 PDT
Comment on attachment 61070 [details]
One more null-check

OK, hope this one is enough.
Comment 7 Gustavo Noronha (kov) 2010-07-12 07:34:42 PDT
Comment on attachment 61070 [details]
One more null-check

Landed as r63082. Now to try and fix this in a better way.