Summary: | Web Inspector: No way to see a path to a root for instances in the Heap Snapshot content view | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||||||||
Component: | Web Inspector | Assignee: | Joseph Pecoraro <joepeck> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Attachments: |
|
Description
Joseph Pecoraro
2016-03-14 17:22:48 PDT
Created attachment 274063 [details]
[PATCH] Proposed Fix
Created attachment 274064 [details]
[IMAGE] Simple Path
Created attachment 274065 [details]
[IMAGE] More Complex Path
The images were taken when I was using an eye icon. That is no longer accurate, I switched to just hovering the object-id, @1234, to get the popover. Created attachment 274066 [details]
[PATCH] Proposed Fix
Rebaselined.
Comment on attachment 274066 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=274066&action=review > Source/WebInspectorUI/UserInterface/Models/HeapSnapshotRootPath.js:120 > + let component = this._canPropertyNameBeDotAccess(propertyName) ? "." + propertyName : "[" + doubleQuotedString(propertyName) + "]"; Slick. > Source/WebInspectorUI/UserInterface/Views/HeapSnapshotInstancesContentView.css:130 > + white-space: pre; Why pre? Comment on attachment 274066 [details] [PATCH] Proposed Fix Clearing flags on attachment: 274066 Committed r198196: <http://trac.webkit.org/changeset/198196> All reviewed patches have been landed. Closing bug. |