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)
Core Graveyard
Installer: XPInstall Engine
Tracking
(Not tracked)
VERIFIED
FIXED
M13
People
(Reporter: dougt, Assigned: samir_bugzilla)
References
Details
See nsInstallProgressDialog::GetCancelStatus(PRBool *_retval).
Reporter | ||
Updated•25 years ago
|
Target Milestone: M10
Reporter | ||
Comment 1•25 years ago
|
||
moving to m10
Reporter | ||
Comment 2•25 years ago
|
||
Setting Priority and Serverity per Jan's email.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Comment 3•25 years ago
|
||
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.
Updated•25 years ago
|
Summary: Progress cancel button always returns false → Progress cancel button doesn't work
Target Milestone: M10 → M11
Updated•25 years ago
|
Assignee: dveditz → sgehani
Comment 5•25 years ago
|
||
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?).
Comment 6•25 years ago
|
||
Pushing this out since new dialog is not in
Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall
Engine
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•25 years ago
|
||
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.
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Comment 9•25 years ago
|
||
Verified in build 1999122023.
Updated•25 years ago
|
Status: VERIFIED → REOPENED
Comment 10•25 years ago
|
||
Sorry, made a mistake and verified the wrong bug.
Updated•25 years ago
|
Resolution: FIXED → ---
Assignee | ||
Updated•25 years ago
|
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 11•25 years ago
|
||
Ummm, like I marked this RESOLVED/FIXED earlier today.
Comment 12•25 years ago
|
||
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.
Assignee | ||
Updated•25 years ago
|
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Assignee | ||
Comment 13•25 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.
Comment 14•25 years ago
|
||
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.
Comment 15•25 years ago
|
||
Build 2000-01-03-09-M13(WIN), 2000-01-03-08-M13(MAC), 2000-01-03-08-M13(LINUX)
Uh, yeah. It's working.
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•