New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 681467 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary movement of icons is seen in Memory section after taking 7 Snapshots

Reported by jshan...@etouch.net, Jan 16 2017

Issue description

Chrome 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
 
Actual_video.mp4
936 KB View Download
Expected_video.mp4
904 KB View Download
Actual_Expected_result.jpg
47.5 KB View Download
Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: alph@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 57.0.2978.0 (Revision: 442756).
Bad build: 57.0.2979.0 (Revision: 443120).

You are probably looking for a change made after 443047 (known good), but no later than 443048 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/57b7617e61e86c83be5819ef826093cafa868f73..b3cc3c0f613ff302f4833781a388e2ad8646f079

@alph -- Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.
Adding RB Label as this is a recent Regression. Please remove if this is not required.
Thank You.
Labels: Needs-triage-Mobile
Labels: -Needs-triage-Mobile
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!
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!
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!

Comment 7 by alph@chromium.org, Feb 9 2017

Cc: kkaluri@chromium.org alph@chromium.org
 Issue 688325  has been merged into this issue.

Comment 9 by alph@chromium.org, Feb 10 2017

Labels: -ReleaseBlock-Stable Merge-Request-57
Project Member

Comment 10 by sheriffbot@chromium.org, Feb 11 2017

Labels: -Merge-Request-57 Hotlist-Merge-Approved Merge-Approved-57
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
Project Member

Comment 11 by bugdroid1@chromium.org, Feb 11 2017

Labels: -merge-approved-57 merge-merged-2987
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

Comment 12 by alph@chromium.org, Feb 11 2017

Status: Fixed (was: Assigned)
Labels: TE-Verified-57.0.2987.54 TE-Verified-M57
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
 
Issue 681467.mp4
941 KB View Download

Comment 14 by alph@chromium.org, Mar 6 2017

Cc: kavvaru@chromium.org durga.behera@chromium.org brajkumar@chromium.org ajha@chromium.org
 Issue 696426  has been merged into this issue.

Sign in to add a comment