255432 - Ctrl-f, Ctrl-g, F3 cannot be used to search for text in a ftp-listing

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[k b]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a3) Gecko/20040809
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a3) Gecko/20040809

Ctrl-f, Ctrl-g cannot be used to search for text in a ftp-listing

they are also grey marked on the Edit menu.

it is possible to use type ahead to find some word once, but then when pressing
Ctrl-g or F3 to find next nothing happens.

Reproducible: Always
Steps to Reproduce:
1. see above 
Actual Results:  
searching more then onece not possible

Expected Results:  
should be possible

Comment 1

[Boris Zbarsky [:bzbarsky]]

Presumably a progress listener issue?

Comment 2

[Stefan Borggraefe]

WFM with a gtk2 trunk build from Aug 10.

Comment 3

[k b]

still not good in:
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a3) Gecko/20040812

i did remove everything in my profile except the prefs.js and the bookmarks.html
file. mozilla is run on fedora core 2 if that makes any difference.

Comment 4

[[e-mail]]

Old versions of Mozilla would show the MIME type as text/html in Page Info but
the trunk now shows it as application/http-index-format - is that intentional?

Comment 5

[Stefan Borggraefe]

(In reply to comment #2)
> WFM with a gtk2 trunk build from Aug 10.

Strike that. I was going over a proxy. :-( I'm now seeing this, too.

Comment 6

[k b]

can this problem have somthing to with the way directory listings are handled,
not just ftp-listing?

the same problem occurs when listing a local directory, ie:
file:///home/user/

Comment 7

[Darin Fisher]

biesi: this sounds like fallout from bug 247607.

Comment 8

[Christian :Biesinger (don't email me, ping me on IRC)]

hmm, why does the stream converter not set a new content type on the channel?
and, if it doesn't, why does anything work? taking for now...

Comment 9

[Christian :Biesinger (don't email me, ping me on IRC)]

Attachment: set content type earlier

answer: content-type setting happens too late.

things worked because this thingy is a document viewer or something, not
directly a stream listener.

Comment 10

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 157349 [details] [diff] [review]
set content type earlier

r=bzbarsky

Comment 11

[Darin Fisher]

Comment on attachment 157349 [details] [diff] [review]
set content type earlier

maybe change

// set ... up

to

// setup ...

?


sr=darin

Comment 12

[Christian :Biesinger (don't email me, ping me on IRC)]

checked in, with comment changed

Checking in xpfe/components/directory/nsDirectoryViewer.cpp;
/cvsroot/mozilla/xpfe/components/directory/nsDirectoryViewer.cpp,v  <-- 
nsDirectoryViewer.cpp
new revision: 1.115; previous revision: 1.114
done