Bug 156431

Summary:

Web Inspector: HeapSnapshot instance property path popover should include a descriptive header

Product:

WebKit

Reporter:

Joseph Pecoraro <joepeck>

Component:

Web Inspector

Assignee:

Joseph Pecoraro <joepeck>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bburg, ggaren, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

All

OS:

All

Attachments:

Description	Flags
[PATCH] Proposed Fix	timothy: review+, bburg: commit-queue-
[IMAGE] Popover in Active Window	none
[IMAGE] Popover in inactive Window	none

Joseph Pecoraro

Reported 2016-04-08 18:39:16 PDT

* SUMMARY HeapSnapshot instance property path popover should include a descriptive header A heading like “shortest property path to @1234” would help the user understand: (a) the rows “window” and “r” indicate a path of “window.r” (b) this is one of potentially many paths, and other paths are not shown

Attachments
[PATCH] Proposed Fix (6.89 KB, patch) 2016-04-08 19:01 PDT, Joseph Pecoraro	timothy: review+ bburg: commit-queue-	Details Formatted Diff Diff
[IMAGE] Popover in Active Window (189.39 KB, image/png) 2016-04-08 19:01 PDT, Joseph Pecoraro	no flags	Details
[IMAGE] Popover in inactive Window (297.27 KB, image/png) 2016-04-08 19:01 PDT, Joseph Pecoraro	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Joseph Pecoraro

Comment 1 2016-04-08 18:39:24 PDT

<rdar://problem/25633594>

Joseph Pecoraro

Comment 2 2016-04-08 19:01:11 PDT

Created attachment 276072 [details] [PATCH] Proposed Fix

Joseph Pecoraro

Comment 3 2016-04-08 19:01:42 PDT

Created attachment 276073 [details] [IMAGE] Popover in Active Window

Joseph Pecoraro

Comment 4 2016-04-08 19:01:55 PDT

Created attachment 276074 [details] [IMAGE] Popover in inactive Window

Joseph Pecoraro

Comment 5 2016-04-11 13:00:51 PDT

http://trac.webkit.org/changeset/199308

Note You need to log in before you can comment on or make changes to this bug.