New issue
Advanced search Search tips

Issue 671449 link

Starred by 2 users

Issue metadata

Status: Archived
Owner: ----
Closed: Sep 13
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Extension uninstall dialog clearup.

Project Member Reported by lgcheng@google.com, Dec 6 2016

Issue description

Current implementation of uninstall dialog is over complicated than necessary to handle the life cycle of dialog-view objects. 
According to msw@
A lot of the delegation and coordination should just disappear by creating a simple dialog (like SimpleMessageBoxViews) that just needs strings, icons, and callbacks (or an ACCEPT/CANCEL/CLOSE enum return value).

I have submitted implementation for Arc apps in https://codereview.chromium.org/2529783002/

But I don't have sufficient experience with "extension world" So I would expect someone who is more familiar with extension system could take over this in-depth refactor/clean up.



 

Comment 1 by ajha@chromium.org, Dec 6 2016

Labels: M-57
Tagging with canary milestone.

Comment 2 by msw@chromium.org, Dec 6 2016

Components: Platform>Extensions UI>Browser
Labels: -OS-All Hotlist-CodeHealth Hotlist-TechnicalDebt OS-Chrome OS-Linux OS-Mac OS-Windows
Cc: rdevlin....@chromium.org catmulli...@chromium.org
Status: Available (was: Untriaged)
Cc: karandeepb@chromium.org
Cc: -catmulli...@chromium.org
Status: Archived (was: Available)
Archiving old bugs that haven't been actively assigned in over 180 days.

If you feel this issue should still be addressed, feel free to reopen it or to file a new issue. Thanks!

Sign in to add a comment