Regression: Unnecessary movement of icons is seen in Memory section after taking 7 Snapshots
Reported by
jshan...@etouch.net,
Jan 16 2017
|
|||||||||
Issue descriptionChrome Version: 57.0.2983.0 (Official Build) (64-bit)266a86c94d34f3c7c3d22ee00624636b801f80d3-refs/heads/master@{#443819}-32/64 bit OS: Windows(7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.11.6, 10.12.1) Steps: 1. Launch Chrome, open Devtools on NTP and dock it bottom. 2. Go to Memory section, select 'Take Heap Snapshot' and record at least 7 Snapshot 3. Observe. Actual: Unnecessary movement of icons (Take heap snapshot, Clear all profiles and Collect Garbage) is seen after taking 7 Snapshots Expected: No such movement of icons (Take heap snapshot, Clear all profiles and Collect Garbage) should be seen after taking 7 Snapshots This is regression issue, broken in ‘M 57’ and will soon update other info : Good build:57.0.2978.0 Bad build: 57.0.2979.0
,
Jan 20 2017
,
Jan 23 2017
,
Jan 30 2017
alph@ Since this issue is marked as RB-Stable and observed in latest canary #58.0.2996.0 Can we get any latest update on this issue? Thanks!
,
Feb 7 2017
Able to reproduce this issue on Ubuntu 14.04 using chrome latest Dev M55-58.0.3000.4 alph@ Ping! Could you please let us know is there any latest update available on this issue? Thanks!
,
Feb 8 2017
A friendly reminder that M57 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 9 2017
,
Feb 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cd03f64dcc04f08e8ed35b058fe92bba1bd9abb1 commit cd03f64dcc04f08e8ed35b058fe92bba1bd9abb1 Author: alph <alph@chromium.org> Date: Fri Feb 10 00:42:42 2017 DevTools: Fix toolbar layout glitches on memory panel BUG= 681467 Review-Url: https://codereview.chromium.org/2682123004 Cr-Commit-Position: refs/heads/master@{#449493} [modify] https://crrev.com/cd03f64dcc04f08e8ed35b058fe92bba1bd9abb1/third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js [modify] https://crrev.com/cd03f64dcc04f08e8ed35b058fe92bba1bd9abb1/third_party/WebKit/Source/devtools/front_end/profiler/profilesPanel.css
,
Feb 10 2017
,
Feb 11 2017
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6fceab2c5c58ea7faa463d15cabd5466749d18ab commit 6fceab2c5c58ea7faa463d15cabd5466749d18ab Author: Alexei Filippov <alph@chromium.org> Date: Sat Feb 11 00:51:20 2017 DevTools: Fix toolbar layout glitches on memory panel BUG= 681467 Review-Url: https://codereview.chromium.org/2682123004 Cr-Commit-Position: refs/heads/master@{#449493} (cherry picked from commit cd03f64dcc04f08e8ed35b058fe92bba1bd9abb1) Review-Url: https://codereview.chromium.org/2685333006 . Cr-Commit-Position: refs/branch-heads/2987@{#458} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} [modify] https://crrev.com/6fceab2c5c58ea7faa463d15cabd5466749d18ab/third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js [modify] https://crrev.com/6fceab2c5c58ea7faa463d15cabd5466749d18ab/third_party/WebKit/Source/devtools/front_end/profiler/profilesPanel.css
,
Feb 11 2017
,
Feb 15 2017
verified this issue on WINDOWS 10, Ubuntu 14.04 and Mac 10.12.3 with chrome beta #57.0.2987.54 Observed no unnecessary movements of icons up to 15 profile screenshots Hence Adding TE-verified labels
,
Mar 6 2017
Issue 696426 has been merged into this issue. |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by msrchandra@chromium.org
, Jan 16 2017Owner: alph@chromium.org
Status: Assigned (was: Unconfirmed)