Bug 21283 - Profiler Crashes When Started
Summary: Profiler Crashes When Started
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P1 Normal
Assignee: Geoffrey Garen
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2008-10-01 13:19 PDT by Carsten Nielsen
Modified: 2008-10-02 09:26 PDT (History)
2 users (show)

See Also:


Attachments
Crash details from the crash report dialog. (23.95 KB, text/plain)
2008-10-01 13:24 PDT, Carsten Nielsen
no flags Details
System configuration from crash report dialog. (1.19 KB, text/plain)
2008-10-01 13:26 PDT, Carsten Nielsen
no flags Details
patch (3.83 KB, patch)
2008-10-02 00:28 PDT, Geoffrey Garen
sam: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carsten Nielsen 2008-10-01 13:19:25 PDT
Using the latest nightly [37126] when attempting to start the Javascript profiler from the web inspector it crashes. There does not seem to be an special circumstances, it happens anytime I try to start it. I am using the web inspector in a separate window.
Comment 1 Carsten Nielsen 2008-10-01 13:24:35 PDT
Created attachment 23990 [details]
Crash details from the crash report dialog.
Comment 2 Carsten Nielsen 2008-10-01 13:26:22 PDT
Created attachment 23991 [details]
System configuration from crash report dialog.
Comment 3 Mark Rowe (bdash) 2008-10-01 14:14:44 PDT
<rdar://problem/6261955>
Comment 4 Kevin McCullough 2008-10-01 16:41:09 PDT
This did not occur in r37056.  I'll see if I can reduce further.
Comment 5 Kevin McCullough 2008-10-01 16:57:23 PDT
From my investigations it looks like 37125 is the culprit.
Comment 6 Geoffrey Garen 2008-10-02 00:28:39 PDT
Created attachment 24010 [details]
patch
Comment 7 Geoffrey Garen 2008-10-02 09:23:08 PDT
Committed revision 37191.
Comment 8 Darin Adler 2008-10-02 09:25:10 PDT
Thanks. My bad!
Comment 9 Darin Adler 2008-10-02 09:26:44 PDT
Things in header files shouldn't be marked "static", tho!