Bug 27146

Summary: JSDebugger crashes after reloading from a breakpoint
Product: WebKit Reporter: Oliver Hunt <oliver>
Component: Web Inspector (Deprecated)Assignee: Oliver Hunt <oliver>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, timothy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
harden the debugger timothy: review+, commit-queue: commit-queue-

Description Oliver Hunt 2009-07-10 01:59:38 PDT 
In certain scenarios it is possible to end up with a dead callframe in the inspector, this leads to a crash in the debugger.
Comment 1 Oliver Hunt 2009-07-10 02:08:03 PDT 
Created attachment 32552 [details]
harden the debugger
Comment 2 Oliver Hunt 2009-07-10 02:36:26 PDT 
Comment on attachment 32552 [details]
harden the debugger

despite fixing the crash weirdness happens in the debugger afterwards
Comment 3 WebKit Commit Bot 2009-09-11 21:56:41 PDT 
Comment on attachment 32552 [details]
harden the debugger

Rejecting patch 32552 from commit-queue.

This patch will require manual commit. Patch https://bugs.webkit.org/attachment.cgi?id=32552 from bug 27146 failed to download and apply.
Comment 4 Eric Seidel (no email) 2009-09-14 09:51:44 PDT 
patching file WebCore/inspector/InspectorController.cpp
Hunk #1 FAILED at 535.
1 out of 1 hunk FAILED -- saving rejects to file WebCore/inspector/InspectorController.cpp.rej
Comment 5 Eric Seidel (no email) 2009-09-18 12:01:19 PDT 
A quick git log search looks like this hasn't been landed yet.
Comment 6 Eric Seidel (no email) 2009-10-05 10:52:49 PDT 
Ping?
Comment 7 Eric Seidel (no email) 2009-10-19 15:13:27 PDT 
This patch has been chillin-like-a-villin in the to-be-committed list for over a month now.  Is the patch still wanted?
Comment 8 Timothy Hatcher 2009-10-19 20:00:05 PDT 
This was actually fixed by: http://trac.webkit.org/changeset/45733

I don't think this full patch is needed, but I landed the parts that are still relevant.

Landed the relevant parts in r49832.