Closed Bug 237510 Opened 20 years ago Closed 20 years ago

installer exits on "pause" and before Connection Dropped dialog can be seen

Categories

(SeaMonkey :: Installer, defect)

All
Linux
defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ajschult784, Assigned: ajschult784)

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

the pause feature while downloading currently doesn't work.  With linux trunk
2004031408, the installer simply exists if you hit pause.  For similar reasons,
the installer exits after trying to put up the dropped connection dialog, which
is supposed to actually wait for the user to hit "cancel" or "resume".

This is probably caused by or at least made worse by the patch from bug 160973.
Attached patch patch (obsolete) — Splinter Review
this patch calls gtk_main on pause and gtk_main_quit on cancel or restore
this removes the bInstallClicked.  it doesn't seem to do anything but screw up
DLResume.

Also, the need to call gtk_main_quit in DLCancel has nothing to do with auto
mode.
Attached patch patch v2Splinter Review
this is the right patch.
I need some sleep.
Attachment #143945 - Attachment is obsolete: true
Attachment #143948 - Flags: superreview?(bryner)
Attachment #143948 - Flags: review?(dveditz+bmo)
nobody noticed this in 1.7a, but it seems like a serious regession that would
frustrate users who encountered it.
Flags: blocking1.7b?
Comment on attachment 143948 [details] [diff] [review]
patch v2

looks ok, I assume you've checked that the gtk_main/gtk_main_quit()s match up
in both the auto and non-auto install cases.
Attachment #143948 - Flags: superreview?(bryner) → superreview+
Comment on attachment 143948 [details] [diff] [review]
patch v2

r=dveditz
Attachment #143948 - Flags: review?(dveditz+bmo) → review+
Comment on attachment 143948 [details] [diff] [review]
patch v2

this prevents the installer from exitting when pressing "pause".
Attachment #143948 - Flags: approval1.7b?
Comment on attachment 143948 [details] [diff] [review]
patch v2

a=chofmann for 1.7 -- wait until later today to check in.
Attachment #143948 - Flags: approval1.7b? → approval1.7+
Flags: blocking1.7b? → blocking1.7b-
fix checked in by Neil.

for the record, I retested the installer before bug 160973, and it had similar
problems to this bug, but they were confined to auto mode.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: