Issue 632917 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	alph@chromium.org
Closed:	Jul 2016
Components:	Platform>DevTools>Memory
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	3
Type:	Bug

Always show all heap snapshot properties

Project Member Reported by alph@chromium.org, Jul 29 2016

Issue description

Currently heap snapshot does not show some internal properties by default.
It may hide essential object relation links. Remove the setting and make all the properties always visible.

Project Member

Comment 1 by bugdroid1@chromium.org, Jul 30 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b13b5d875d636d36617943e2dfb2dcf2831cc28a

commit b13b5d875d636d36617943e2dfb2dcf2831cc28a
Author: alph <alph@chromium.org>
Date: Sat Jul 30 04:04:51 2016

DevTools: Remove Advanced heap snapshot properties setting.

Make it always show all the object properties.

BUG= 632917 

Review-Url: https://codereview.chromium.org/2188413004
Cr-Commit-Position: refs/heads/master@{#408859}

[modify] https://crrev.com/b13b5d875d636d36617943e2dfb2dcf2831cc28a/third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-test.js
[modify] https://crrev.com/b13b5d875d636d36617943e2dfb2dcf2831cc28a/third_party/WebKit/Source/devtools/front_end/heap_snapshot_worker/HeapSnapshotLoader.js
[modify] https://crrev.com/b13b5d875d636d36617943e2dfb2dcf2831cc28a/third_party/WebKit/Source/devtools/front_end/heap_snapshot_worker/JSHeapSnapshot.js
[modify] https://crrev.com/b13b5d875d636d36617943e2dfb2dcf2831cc28a/third_party/WebKit/Source/devtools/front_end/profiler/HeapSnapshotProxy.js
[modify] https://crrev.com/b13b5d875d636d36617943e2dfb2dcf2831cc28a/third_party/WebKit/Source/devtools/front_end/profiler/module.json

Comment 2 by alph@chromium.org, Jul 30 2016

Status: Fixed (was: Assigned)

►

Issue 632917 link

Issue metadata

Always show all heap snapshot properties

Issue description

Comment 1 by bugdroid1@chromium.org, Jul 30 2016

Comment 1 Deleted

Comment 2 by alph@chromium.org, Jul 30 2016

Comment 2 Deleted

Sign in to add a comment