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

Issue 900554 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 16
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Chrome extension uninstall URL not working in all cases

Reported by shaneg.l...@gmail.com, Oct 31

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.77 Safari/537.36

Steps to reproduce the problem:
1. Set an uninstall url in the extension manifest using chrome.runtime.setUnintallURL
2. Install the extension from the Chrome Webstore
3. Reload the Chrome Webstore window, note main button now changes from 'Add To Chrome' to 'Remove From Chrome'.
4. Click 'Remove From Chrome'

What is the expected behavior?
The uninstall URL should open.

What went wrong?
Extension is removed but the uninstall URL does not open. This works as intended if the extension is removed from the browser interface.

Did this work before? Yes 

Does this work in other browsers? N/A

Chrome version: 70.0.3538.77  Channel: stable
OS Version: 10.0
Flash Version:
 
Components: Platform>Extensions
Labels: Needs-Bisect Needs-Triage-M70
Cc: viswa.karala@chromium.org
Labels: Triaged-ET Needs-Feedback
Thanks for filing the issue!

@Reporter: If possible could you please provide sample Test File/URL that reproduces the issue which help in further triaging the issue in better way.

Thanks!
Try with this extension:
https://chrome.google.com/webstore/detail/color-up-your-tab-–-color/aadacekhkmgphglhdkholnompgfbpnkl
Note that if you remove it through the browser interface it opens an uninstall page, but if you remove it from the Chrome Store page, it does not. 
Project Member

Comment 5 by sheriffbot@chromium.org, Nov 1

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Pri-2 -Needs-Bisect hasbisect-per-revision RegressedIn-67 Target-70 Target-71 Target-72 FoundIn-72 M-72 FoundIn-71 FoundIn-70 OS-Linux OS-Mac Pri-1
Owner: bdea@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on reported chrome version 70.0.3538.77 and on latest chrome# 72.0.3598.0 using Windows-10, Mac 10.12.6 & Ubuntu 17.10, hence providing Bisect Info

Bisect Info:
================
Good build: 67.0.3390.0
Bad build: 67.0.3393.0

You are probably looking for a change made after 548668 (known good), but no later than 548669 (first known bad).
https://chromium.googlesource.com/chromium/src/+log/f6d55b110ae7ea1a49c131d647f279e8a57d1813..7264a04b6a618b8dd8dfa898940ee2e2e1f68483
Change-Id: I73c1e53ffaadcddd930e34032db2acda6cb9867f
Reviewed-on: https://chromium-review.googlesource.com/966485

@Bettina Dea: Please confirm the issue and help in re-assigning if it is not related to your change.

Thanks!
Status: Started (was: Assigned)
Project Member

Comment 8 by bugdroid1@chromium.org, Nov 16

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

commit b6128ffaa5f2f57e9129c4825b5aa0e6a1b283c9
Author: Bettina <bdea@chromium.org>
Date: Fri Nov 16 21:04:26 2018

Open the uninstall url after extension uninstall on CWS.

We created another uninstall reason when uninstall functionality
was made available in Chrome Web Store but didn't update the
condition for showing the uninstall URL.

I've re-enabled the browser test for
EnsureExtensionUninstallURLIsActiveTabAfterUninstall since I've
run the test 100 times in one run and 600 times in another run and
it did not fail.

Bug:  900554 
Change-Id: I805840717709606a8e98d1c82cbea2b611efdfd6
Reviewed-on: https://chromium-review.googlesource.com/c/1336522
Commit-Queue: Bettina Dea <bdea@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608937}
[modify] https://crrev.com/b6128ffaa5f2f57e9129c4825b5aa0e6a1b283c9/chrome/browser/ui/views/extensions/extension_uninstall_dialog_view_browsertest.cc
[modify] https://crrev.com/b6128ffaa5f2f57e9129c4825b5aa0e6a1b283c9/extensions/browser/api/runtime/runtime_api.cc

Status: Fixed (was: Started)

Sign in to add a comment