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

Issue 922946 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jan 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug-Regression



Sign in to add a comment

Regression: Unwanted extra space is seen on App info option in App menu

Project Member Reported by khush...@virtusa.com, Jan 17 (5 days ago)

Issue description

Chrome Version: 73.0.3673.0 (Official Build) Revision e5cdf80fb8b493b3be23fc5b7bfac5278aa83225-refs/branch-heads/3673@{#1} (32/64 bit)
OS: Windows (7, 8, 8.1, 10), Mac (10.13.1, 10.13.6, 10.14.3) and Linux(14.04 LTS)

URL: https://chrome.google.com/webstore/detail/pdf-viewer/oemmndcbldboiebfnladdacbdfmadadm

What steps will reproduce the problem?
(1) Launch chrome, navigate to above URL and click on ADD TO CHROME button.
(2) Open any PDF file (ex.http://www.orimi.com/pdf-test.pdf) and click on 'Create shortcut' from wrench menu.
(3) Launch PDF file into separate app window and click on three dot menu and observe menu.

Actual: Unwanted extra space is seen on App info option in App menu.

Expected: No such extra space should seen.

This is a regression issue, broken in 'M70', below is bisect info:

Good Build: 70.0.3534.0
Bad Build:  70.0.3535.0

Using Per-revision script, below is the bisect info:

You are probably looking for a change made after 586177 (known good), but no later than 586178 (first known bad).

CHANGELOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.

https://chromium.googlesource.com/chromium/src/+log/b54f8287c4a6d395b5406b7f3aa1495584f7b3d0..a0ec9e44d5dcc4e6696edd5fec7bfe2b79ef4c23

Suspect: https://chromium.googlesource.com/chromium/src/+/a0ec9e44d5dcc4e6696edd5fec7bfe2b79ef4c23

@alancutter: Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

NOTE: Issue is also seen on M-71 Stable #71.0.3578.98, M-72 Beta #72.0.3626.64, and M-73 Canary #73.0.3674.0

Please refer the attached screen-cast.

Thank You..!!


 
Actual_Space.mp4
1.0 MB View Download
Expected_Space.mp4
1.4 MB View Download

Comment 1 by thestig@chromium.org, Jan 17 (5 days ago)

Components: -Internals>Plugins>PDF
Please do not use the Internals>Plugins>PDF label for https://chrome.google.com/webstore/detail/pdf-viewer/oemmndcbldboiebfnladdacbdfmadadm - it is not the Chrome PDF Viewer. 

Comment 2 by alancutter@chromium.org, Jan 17 (5 days ago)

Components: -Platform>Apps UI>Browser>WebAppInstalls
Labels: -OS-Linux -OS-Windows -Pri-1 -OS-Mac Pri-3
Status: WontFix (was: Assigned)
Looks like the menu has two columns one for the main item text and one for the minor item text (the origin in this case) such that they do not overlap. The extra gap came from the addition of the "Uninstall <app name>..." menu entry pushing the first column boundary to the right.

It's a bit bizzare to be able to create a bookmark out of a chrome extension but I guess it follows naturally from the extension having a URL. I'm a bit surprised we support creating bookmark apps with the chrome-extension:// protocol but I can't think of anything particularly broken about this feature other than the crazy long origin text.

Sign in to add a comment