Bug 156627

Summary: Web Inspector: Events in JavaScript & Events timeline have no profile children
Product: WebKit Reporter: Timothy Hatcher <timothy>
Component: Web InspectorAssignee: Matt Baker <mattbaker>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Screenshot
none
Screenshot of Call Trees View
none
[Patch] Proposed Fix none

Timothy Hatcher
Reported 2016-04-15 09:34:52 PDT
Created attachment 276476 [details] Screenshot A simple page like daring fireball.net shows unload has children, but expanding it does not show anything. A (program) earlier is the same. See screenshot recorded during page load. Either these are so fast they have no children and we should not show the disclosure arrow. Or we are missing profile data here.
Attachments
Screenshot (109.24 KB, image/png)
2016-04-15 09:34 PDT, Timothy Hatcher 		no flags
Details
Screenshot of Call Trees View (79.28 KB, image/png)
2016-04-15 09:37 PDT, Timothy Hatcher 		no flags
Details
[Patch] Proposed Fix (4.99 KB, patch)
2016-04-24 14:32 PDT, Matt Baker 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2016-04-15 09:36:18 PDT
<rdar://problem/25749740>
Timothy Hatcher
Comment 2 2016-04-15 09:37:03 PDT
Created attachment 276477 [details] Screenshot of Call Trees View This is a bug in the Events view, since the Call Trees view has samples for other functions under unload.
Joseph Pecoraro
Comment 3 2016-04-15 10:09:24 PDT
<rdar://problem/25275724>
Matt Baker
Comment 4 2016-04-24 14:32:22 PDT
Created attachment 277197 [details] [Patch] Proposed Fix
WebKit Commit Bot
Comment 5 2016-04-24 17:59:24 PDT
Comment on attachment 277197 [details] [Patch] Proposed Fix Clearing flags on attachment: 277197 Committed r199971: <http://trac.webkit.org/changeset/199971>
WebKit Commit Bot
Comment 6 2016-04-24 17:59:28 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.