141810 - Save Page As Text Fails, Pegs CPU, Brings System to Its Knees

Status: VERIFIED DUPLICATE of bug 141455

(Core Graveyard :: File Handling, defect)

Description

[Felix Miata]

2002050108 OS/2 trunk

To reproduce:
1-Open any web page (I use Google)
2-Open a new tab
3-Open a page (I use an ASP search result)
4-Select 'File -> Save Page As'
5-In filepicker (I run XFile) choose type "text file" and a suitable filename
6-Select "OK"

Actual results:
1-CPU meter pegs
2-Mozilla is no longer responsive
3-File saving dialog does not open
4-System speed seems to drop to 0.1 Mhz
5-Mozilla must be killed to get use of system back
6-Zero byte file created

Expected results:
1-File saving dialog opens
2-File is saved
3-Mozilla remains usable
4-System response remains normal without killing Mozilla

Additional information:
If I instead choose "web page, HTML only", the file is saved, but the download
manager history window comes up instead of the normal saving progress dialog.
Text file save fails the same on both HPFS and FAT16.

Comment 1

[Boris Zbarsky [:bzbarsky]]

The download manager thing is a separate issue (that's the new default behavior).

I cannot reproduce this on Linux... Could you please list the exact urls you are
using?

Comment 2

[Felix Miata]

How do you change from the new default behavior to the old behavior?

URL's don't matter, but these suit the purpose:
Tab 1 - http://www.google.com/
Tab 2 - 
     Initial - http://us.imdb.com/search
     Results page to save - http://us.imdb.com/Name?Dunst,+Kirsten

Comment 3

[Mike Kaply [:mkaply]]

Confirm on OS/2. 

Checking.

Comment 4

[Boris Zbarsky [:bzbarsky]]

> How do you change from the new default behavior to the old behavior?

See Bug 121324, comment 9.  The UI is bug 141278 or bug 132440.

This looks like it's OS/2-only....

Comment 5

[Mike Kaply [:mkaply]]

Broke between 29th and 30th.

My file picker changes went in there, but they also went into the branch and the 
branch is not hanging.

I'm going to try backing out my file picker changes and see what happens.

Comment 6

[Mike Kaply [:mkaply]]

hangs without my filepicker changes as well...

Comment 7

[Felix Miata]

Worksforme in 2002050208 OS/2 trunk.

Comment 8

[Felix Miata]

In prefs.js I put user_pref("browser.downloadmanager.behavior", 1);. I got all
the (preferred) old behavior back now.

Comment 9

[Mike Kaply [:mkaply]]

Man I love to see things like this fixed with no help from me.

I worry that Os/2 is horribly broke when I see this stuff.

*** This bug has been marked as a duplicate of 141455 ***

Comment 10

[Felix Miata]

Out of extreme curiosity, how did you make the connection to bug 141455? Lack of
any other checkins that could have produced a fix? Any connection between file
save hang and select text hang beyond the word hang seems rather tenuous.

Comment 11

[Mike Kaply [:mkaply]]

when I looked at the checkins that could have caused this, I noticed a checkin 
to nsPlainTextSerializer.

nsPlainTextSerializer is the code that converts files to TXT.

Then when I looked at the time frame in which the bug was fixed, I noticed 
another checkin to nsPlainTextSerializer that specifically related to fixing a 
bug in the code I had seen checked in before, and it specifically referenced a 
hang.

So I concluded that what you were seeing was just another manifestation of that 
bug.

Comment 12

[sairuh (rarely reading bugmail)]

marking verified as a duplicate.

if you decide to reopen this bug, please clarify why.

search string for bugspam removal: SalviaGuaranitica