New issue
Advanced search Search tips

Issue 906000 link

Starred by 1 user
Status: WontFix
Owner:
rdevlin....@chromium.org
Closed: Nov 16
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
jawag@chromium.org
bklmn@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary 'PDF Viewer' extension icon gets enabled for any google search.

Reported by db...@etouch.net, Nov 16 
Chrome Version: 72.0.3612.0 Revision 15e37528e073fa06eb4609e293cadd2afee1021e-refs/branch-heads/3612@{#1}(32/64 bit)
OS: Windows(7,8,8.1,10), Mac(10.13.1, 10.13.6, 10.14.2), 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) After adding 'PDF Viewer' extension, open NTP and search for any keyword and observe 'PDF Viewer' icon.

Actual: Unnecessary 'PDF Viewer' extension icon gets enabled for any google search.

Expected: 'PDF Viewer' extension icon should not gets enabled for any google search.

This is a regression issue,broken in 'M72' will soon update the other info:

Good build:72.0.3599.0(Revision: 604792)
Bad Build: 72.0.3601.0(Revision: 605195)
Actual_Icon.mp4
405 KB View Download
Expected_Icon.mp4
667 KB View Download

Comment 1 by db...@etouch.net, Nov 16

Labels: hasbisect
Owner: rdevlin....@chromium.org
Status: Assigned (was: Unconfirmed)
Chrommium Bisect:
https://chromium.googlesource.com/chromium/src/+log/74121ab3d156d9152da03f484fa07b6b1bbbe845..1dd1e22adee58810980d4844d8d7318ed42d7f1c?pretty=fuller&n=100

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

@rdevlin.cronin: 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.

Thank you..!

Comment 2 by rdevlin....@chromium.org, Nov 16

Cc: jawag@chromium.org bklmn@chromium.org
Status: WontFix (was: Assigned)
This is WAI.  Beginning with RuntimeHostPermissions, we show the icon as enabled if it can either be clicked OR if it is actively running (or can run) on the given page.  The PDFViewer extension runs on every page by default.  The reason it's greyed out on the NTP is because the NTP is a protected page.

Comment 3 by thestig@chromium.org, Nov 16

Components: -Internals>Plugins>PDF
PDFViewer extension != Chrome PDF Viewer.
►

Sign in to add a comment