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

Issue 611703 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:'Manage shortcuts' link is misplaced for installation bubble of extension.

Reported by dmascare...@etouch.net, May 13 2016

Issue description

Chrome Version: 52.0.2735.0 Revision 9d255f0ca75efea1773fcb718c368a7a6bc4bba4-refs/heads/master@{#393409}(32/64 bit)
OS: Windows (7,8,10), Linux (14.04 LTS)

URL:https://chrome.google.com/webstore/detail/screencastify-screen-vide/mmeijimgabbpbgpdklnllpncmdofkcpn?hl=en

What steps will reproduce the problem?
(1) Launch chrome, navigate to above url and click on 'ADD TO CHROME' 
(2) Observe the installation bubble. 

Actual:'Manage shortcuts' link is been misplaced within bubble.
Expected:'Manage shortcuts' link should not be misplaced.

This is a regression issue, broken in 'M-52', below is bisect info:

Good Build: 52.0.2719.0
Bad Build: 52.0.2720.0

Narrow Bisect:
https://chromium.googlesource.com/chromium/src/+log/aacdfd0e17cb1597dd22745f73b172a0e2797359..e49d943e9f5f90411313e64d0ae6b646edc85043?pretty=fuller&n=100

Suspecting: r390262

Note: Issue is not seen on Mac OSX(10.10.5, 10.11.4).

 
manage.png
44.8 KB View Download
Labels: ReleaseBlock-Stable
Marking the above issue as RB-Stable as this is a recent regression.

Thank you!
Project Member

Comment 2 by bugdroid1@chromium.org, May 14 2016

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

commit 133b4c13cac797ad830ea23cd6e1e3904aa79a00
Author: estade <estade@chromium.org>
Date: Sat May 14 01:49:50 2016

Fix layout of manage shortcuts link in extension installed bubble.

BUG= 611703 

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

[modify] https://crrev.com/133b4c13cac797ad830ea23cd6e1e3904aa79a00/chrome/browser/ui/views/extensions/extension_installed_bubble_view.cc

Labels: TE-Verified-52.0.2738.0 TE-Verified-M52
Tested the issue on Windows 7, Ubuntu 14.04 using latest M-52(52.0.2738.0). Observed that 'Manage shortcuts' link is not misplaced.
Please find attached screencast.

Marking it as TE-Verified.

611703.mp4
1.0 MB Download

Comment 4 by est...@chromium.org, May 16 2016

Status: Fixed (was: Assigned)

Sign in to add a comment