Closed Bug 913827 Opened 11 years ago Closed 10 years ago

HTTP cache v2: let FTP use the new cache or drop cache use in it completely

Categories

(Core :: Networking: Cache, defect)

Product:
Core
Component:
Networking: Cache
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: mayhemer, Assigned: dragana)

References

Details

Attachments

(1 file)

bug_913827_v1.patch
24.08 KB, patch
michal
: review+
Details | Diff | Splinter Review
To makes things simpler, I vote for just dropping it completely.
Blocks: 913828
We don't care at all about FTP performance (and IIRC we were caching only for directory listings, and browsing FTP that way is even rarer than clicking on an FTP download). So yes, let's disable cache usage.
Blocks: 1061898
No longer blocks: 1061898
Where's the FTP stuff living?
(In reply to Florian Bender from comment #2) > Where's the FTP stuff living? netwerk/protocol/ftp
Assignee: nobody → dd.mozilla
Attachment #8570192 - Flags: review?(michal.novotny)
Attachment #8570192 - Flags: review?(michal.novotny) → review+
Keywords: checkin-needed
That patch didn't have the bug number in the commit message, and I somehow managed to mistype it when I added it myself. https://hg.mozilla.org/integration/mozilla-inbound/rev/4829b02a11af
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: