Closed Bug 633666 Opened 14 years ago Closed 14 years ago

PathifyURI breaks caching of packed XPI js components

Categories

(Core :: XPConnect, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- -

People

(Reporter: mwu, Assigned: mwu)

References

Details

(Keywords: regression)

Attachments

(1 file)

Attached patch Ignore return code from GetHost (deleted) — Splinter Review
PathifyURI checks the return value of GetHost. GetHost is not always implemented, particularly on jar uris. JAR uris are very common when loading js components from packed XPIs. This results in js components from packed extensions not getting cached. Appears to be a regression from bug 609785.
Attachment #511893 - Flags: review?(benjamin)
blocking2.0: --- → ?
OS: Linux → All
Hardware: x86_64 → All
Attachment #511893 - Flags: review?(benjamin) → review+
What's the impact of them not being cached? Correctness? Performance in some cases? This has review, so let's get it in, but if it bounces I think it bounces for good.
blocking2.0: ? → -
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: