Bug 74255

Summary: reproducible crash after closing web inspector and browser window (iCab)
Product: WebKit Reporter: trinler
Component: Web Inspector (Deprecated)Assignee: Pavel Feldman <pfeldman>
Status: RESOLVED FIXED    
Severity: Normal CC: pfeldman, simon.fraser, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac (Intel)   
OS: OS X 10.6   
Attachments:
Description Flags
iCab crashlogs
none
One crash log from the archive
none
Patch
none
[Patch] for ews yurys: review+

Description trinler 2011-12-11 07:20:44 PST
Created attachment 118706 [details]
iCab crashlogs

steps to reproduce crash:

- open any (even empty) page in iCab
- open web inspector (no matter if attached or not)
- close web inspector
- close window
=> crash

crashing started with r101238. up to r101192 no crash happened.
crash does not happen with Safari/WebKit, only with iCab/WebKit, but developer states it seems to be a webkit-bug.

some crashlogs attached.
Comment 1 Mark Rowe (bdash) 2011-12-11 14:16:26 PST
Created attachment 118712 [details]
One crash log from the archive
Comment 2 Pavel Feldman 2011-12-12 10:46:27 PST
Is it repeatable with the tip of the tree WebKit? It might be a WebKit bug in a sense that the way embedder interacts with the WebKit might have changed. In any case, having it working in Safari suggests that the embedder is doing something wrong.
Comment 3 trinler 2011-12-13 05:44:30 PST
It is repeatable with any WebKit revision since r101238 including latest r102661. If I close the Browserwindow without closing the WebInspector first, no crash happens. If I close the WebInspector, I can surf the web inside the window without problems until I close the window, then it crashes. So it seems like the window-close-function tries to access something which already disappears when closing the inspector. Cannot say who does somethning wrong or if an API change happened between r101192 and r101238.
Comment 4 Pavel Feldman 2011-12-19 10:14:22 PST
Created attachment 119880 [details]
Patch
Comment 5 Pavel Feldman 2011-12-19 11:43:18 PST
Created attachment 119894 [details]
[Patch] for ews
Comment 6 Pavel Feldman 2011-12-20 02:03:08 PST
Committed r103314: <http://trac.webkit.org/changeset/103314>
Comment 7 trinler 2011-12-20 11:17:12 PST
Confirmed. Issue is resolved in r103339. Thank you very much.