FTP does not work if memory and disk cache sizes are zero

VERIFIED FIXED in mozilla0.9.9

Status

()

P1
normal
VERIFIED FIXED
17 years ago
16 years ago

People

(Reporter: sfraser_bugs, Assigned: bbaetz)

Tracking

({regression})

Trunk
mozilla0.9.9
PowerPC
Mac System 9.x
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
If you set both memory and disk cache to zero, quit and restart the browser, and 
they try to do FTP, it fails to show any FTP results. FTP just doesn't work at 
all.

I didn't try different memory/disk cache settings.

Comment 1

17 years ago
that is not good.  It use to work.  We need to fix this.  Bradley, can you look
at this soon.  If not, punt to me.
Target Milestone: --- → mozilla0.9.9
(Reporter)

Comment 2

17 years ago
It seems that both memory and disk cache sizes need to be 0 to see this.
(Assignee)

Comment 3

17 years ago
I appear to have local changes which accidentally disable the cache (oops)

I'll play with this tomorrow. Looking at code, I think I just need to ignore
errors from AsyncOpenCacheEntry. Thats what http does, at least...
Keywords: regression
Priority: -- → P1
(Assignee)

Comment 4

17 years ago
Created attachment 67052 [details] [diff] [review]
patch

Comment 5

17 years ago
Comment on attachment 67052 [details] [diff] [review]
patch

sr=darin, one day you might want to modify FTP to use OpenCacheEntry followed
by AsyncOpenCacheEntry only if the cache entry is busy.
Attachment #67052 - Flags: superreview+

Updated

17 years ago
Attachment #67052 - Flags: review+
(Assignee)

Comment 6

17 years ago
Checked in. darin, I'll do that as part of some other bugs I have on fixing up
the cache flags, possibly.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 7

17 years ago
simon: is this working for you now?
Keywords: verifyme

Comment 8

16 years ago
VERIFIED: Mozilla 1.1, Mac OS X.

I'll check the other plats when I run my ftp functional tests w/ caches off.
Status: RESOLVED → VERIFIED
Keywords: verifyme
Whiteboard: checkwin checklinux

Comment 9

16 years ago
VERIFIED: win and linux, Mozilla 1.4b
Whiteboard: checkwin checklinux
You need to log in before you can comment on or make changes to this bug.