Closed Bug 10205 Opened 25 years ago Closed 25 years ago

Progress cancel button doesn't work

Categories

(Core Graveyard :: Installer: XPInstall Engine, defect, P2)

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: dougt, Assigned: samir_bugzilla)

References

Details

See nsInstallProgressDialog::GetCancelStatus(PRBool *_retval).
Target Milestone: M10
moving to m10
Setting Priority and Serverity per Jan's email.
Status: NEW → ASSIGNED
Polling the dialog is probably not the way we want to do it, thus we would remove this function entirely. One alternative is to implement an nsIDOMDocumentObserver to call us when something has changed (but it might call us everytime *we* changed the dlg, which would be bad!), or pass in our own interface purely for use by the cancel button.
Assignee: cathleen → dveditz
reassign to Dan, per today's meeting.
Summary: Progress cancel button always returns false → Progress cancel button doesn't work
Target Milestone: M10 → M11
Assignee: dveditz → sgehani
Samir is now working on this. nsXPInstallManager needs to support a private (nsPIXPI..) interface that allows the dialog to poke the cancel value into the manager. The manager should then stop the downloads if we're in that phase (send errors to Necko?) and if during an install should send signals back to the install script (via the notifier functions?).
Depends on: 7753
Target Milestone: M12 → M13
Pushing this out since new dialog is not in
Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall Engine
Status: NEW → ASSIGNED
Blocks: 22176
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Fix should surface in today's builds. You can cancel during download. Once teh download phase completes and the install phase starts the cancel button is disabled. This is a design decision to accomodate multitriggers.
Status: RESOLVED → VERIFIED
Verified in build 1999122023.
Status: VERIFIED → REOPENED
Sorry, made a mistake and verified the wrong bug.
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 25 years ago25 years ago
Resolution: --- → FIXED
Ummm, like I marked this RESOLVED/FIXED earlier today.
Status: RESOLVED → REOPENED
Build 2000-01-03-09-M13(WIN), 2000-01-03-08-M13(MAC), 2000-01-03-08-M13(LINUX) Cancel button automatically appears disabled from progress dialog for all platforms.
Status: REOPENED → RESOLVED
Closed: 25 years ago25 years ago
Per my comment you can only cancel during download, not during install. You need to download something large. The cancel button will be enabled during download. As soon as extraction of files begins the cancel button is disabled. This is by design: during a multitrigger, say of a.xpi, b.xpi, and c.xpi, respectively, once a.xpi upgrades software package a, and you are in the middle of b.xpi, if you cancel during install there is no way to revert the upgrade action for a.xpi. Thus, our design decision after some deliberation was to disable teh cancel button once extraction begins. Resolving bug as fixed again.
Note that the dialog heading will change from "Downloading" to "Installing" at the point where the cancel button becomes inoperable in the future, to signal the change in state. We will review this with the UE guys, perhaps the cancel button needs to disappear rather than get disabled.
Status: RESOLVED → VERIFIED
Build 2000-01-03-09-M13(WIN), 2000-01-03-08-M13(MAC), 2000-01-03-08-M13(LINUX) Uh, yeah. It's working.
No longer blocks: 22176
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.