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

Issue 666272 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Apps icon get stuck after draging it on chrome://apps

Reported by dmascare...@etouch.net, Nov 17 2016

Issue description

Chrome Version:56.0.2923.0 (Official Build) 455f3cce3da21f8e677100b0c0abd8d35e24876e-refs/heads/master@{#432704}
OS:  Mac (10.11.6, 10.12.1)

What steps will reproduce the problem?
1. Launch chrome and navigate to chrome://apps/
2. Drag any app icon and observe.

Actual:Apps icon get stuck after dragging it.
Expected: App icon should not get stuck.

This is regression issue, broken in 'M 56' and will soon update the info

 
Manual bisect info:
Good build:56.0.2922.0
Bad build:56.0.2923.0

Note: Above issue is not seen on Linux OS and will soon update the Windows OS info.
Actual_app.mov
1009 KB Download
Cc: kkaluri@chromium.org
Labels: hasbisect-per-revision
Owner: paulmeyer@chromium.org
Status: Assigned (was: Unconfirmed)
Bisect Info:
===========

Good build : 56.0.2922.0,  Revision Range 432511
Bad build  : 56.0.2923.0,  Revision Range 432704

After executing the per-revision-bisect script, i got the following CL's between good and bad build versions
===========================================
https://chromium.googlesource.com/chromium/src/+log/2ee91ab48f5ad741bb6889a13fdaffdd0c384cc3..c82b174cd84e9749ec642bfd5bc7805a41d6af0b

The suspecting Change Log is :
-----------
https://chromium.googlesource.com/chromium/src/+/c82b174cd84e9749ec642bfd5bc7805a41d6af0b

From the above CL suspecting the below change
---------------------------
https://codereview.chromium.org/2505113002


paulmeyer@- Could you please look into this issue, if it's related to your change?  if not could you please help us to reassign this issue to the right owner.

Labels: ReleaseBlock-Stable
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.
Cc: tapted@chromium.org
+tapted for mac apps stuff. 
Status: Fixed (was: Assigned)
This was fixed by https://codereview.chromium.org/2509933002/.
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-56; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-56 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD
No Merge needed. Fix is already in.
Project Member

Comment 8 by sheriffbot@chromium.org, Dec 16 2016

Labels: Merge-Review-56 Hotlist-Merge-Review
This bug requires manual review: No test file found in commits.
Please contact the milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), gkihumba@(cros), bustamante@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 9 by sheriffbot@chromium.org, Dec 16 2016

This bug requires manual review: No test file found in commits.
Please contact the milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), gkihumba@(cros), bustamante@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 10 by dimu@google.com, Dec 16 2016

Labels: -Merge-Review-56 -Hotlist-Merge-Review
[Automated comment] removing mislabelled Merge-Review-56, Hotlist-Merge-Review

Sign in to add a comment