Display bugs with memory tab; heap profile snapshot views
Reported by
macon...@mtu.edu,
Jul 3 2017
|
|||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.115 Safari/537.36 Steps to reproduce the problem: 1. Record an 'allocation timeline' on a page 2. Do some things on the page to get arrays/data into the timeline 3. Stop recording and look at the results What is the expected behavior? To be able to look through the data What went wrong? If you open and collapse a drop down for an array or something in the profile view, the next time you open it is only white space and the data is gone. If you scroll, it will sometimes appear but randomly scrolls the screen down Did this work before? N/A Chrome version: 59.0.3071.115 Channel: stable OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version: Applies to any dropdowns in the profile, either outer layer or within other dropdowns
,
Jul 4 2017
Tested the issue on windows 7 & 10 using chrome M59 #59.0.3071.115 and M61 #61.0.3147.0 and issue is not reproduced. Attached screencast for reference. @maconard-- Could you please check attached screencast and confirm us if we had missed any steps in reproducing the issue and also please try in fresh chrome profile without any extensions and flags enabled. Thanks!
,
Jul 4 2017
Issue 738931 has been merged into this issue.
,
Jul 10 2017
I can reproduce it. Thanks for the report!
,
Jul 10 2017
,
Jul 10 2017
Bisected to the culprit change: https://chromium.googlesource.com/chromium/src/+/8309530bfed7acbabd6a64adc53c5c7f2102e745
,
Jul 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/83bf8907db8938eb97d2d16fe6cf2c347727b739 commit 83bf8907db8938eb97d2d16fe6cf2c347727b739 Author: Alexei Filippov <alph@chromium.org> Date: Thu Jul 13 05:35:43 2017 DevTools: Fix memory panel datagrid being empty. 1. The revealed getter should not return true for detached nodes. 2. The height of a node used to calculate the viewport should not depend on the node is attached or not. Bug: 738932 Change-Id: Ib526c3476fd51996158d87736bdeb6cf2678aed5 Reviewed-on: https://chromium-review.googlesource.com/567774 Reviewed-by: Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#486282} [add] https://crrev.com/83bf8907db8938eb97d2d16fe6cf2c347727b739/third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-summary-expand-collapse-expected.txt [add] https://crrev.com/83bf8907db8938eb97d2d16fe6cf2c347727b739/third_party/WebKit/LayoutTests/inspector/profiler/heap-snapshot-summary-expand-collapse.html [modify] https://crrev.com/83bf8907db8938eb97d2d16fe6cf2c347727b739/third_party/WebKit/Source/devtools/front_end/data_grid/DataGrid.js [modify] https://crrev.com/83bf8907db8938eb97d2d16fe6cf2c347727b739/third_party/WebKit/Source/devtools/front_end/profiler/HeapSnapshotDataGrids.js
,
Jul 13 2017
,
Jul 13 2017
,
Jul 13 2017
This bug requires manual review: We are only 11 days from stable. Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 15 2017
Since it's quite late and we're only a few weeks away from M60 stable, the bar is much higher for merges since we're trying to minimize risk and ensure high stability. My recommendation is to wait until M61. This doesn't seem like a critical bug, that is impacting large number of users. Rejecting merge. If you think otherwise, please re-apply Merge-Request label with explanation.
,
Jul 19 2017
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by macon...@mtu.edu
, Jul 3 20179.8 KB
9.8 KB View Download