Open
Bug 315350
Opened 19 years ago
Updated 2 years ago
Need URI helpers
Categories
(Toolkit :: UI Widgets, defect)
Toolkit
UI Widgets
Tracking
()
NEW
People
(Reporter: Gavin, Unassigned)
Details
Attachments
(1 file, 2 obsolete files)
31.00 KB,
patch
|
Details | Diff | Splinter Review |
For creating, verifying scheme, etc. This is done multiple places in-tree in many different ways, and could benefit from having a consistent and stable set of helpers. (This is not a widgets bug, there's no better component)
Reporter | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Comment 1•19 years ago
|
||
I'm just dumping what I have so far in case anyone wants to comment.
Comment 2•19 years ago
|
||
Drive by comment: getSchemeFromString could use nsIIOService::extractScheme.
Reporter | ||
Comment 3•19 years ago
|
||
Patch that makes browser.js use some of the introduced helpers. I think some of the provided functions (e.g. getBaseURL) won't be used very much, so I think I'll end up removing them. Also fixes getSchemeFromString per Darin's comment. I've barely tested this, it's probably buggy.
Attachment #202865 -
Attachment is obsolete: true
Reporter | ||
Comment 4•19 years ago
|
||
Attachment #203223 -
Attachment is obsolete: true
Reporter | ||
Updated•19 years ago
|
Priority: -- → P1
Reporter | ||
Updated•18 years ago
|
Priority: P1 → P2
Reporter | ||
Comment 5•18 years ago
|
||
Ways of dealing with nsIURI/URI strings sounds like something that should probably be considered for inclusion in FUEL at some point, the stuff in the patch here may be a good starting point.
Comment 6•17 years ago
|
||
Bug 380813 adds | IO.newURI | Can the IO code be used or extended a bit to support the URI helpers? Perhaps the "offline" code in your patch could be moved to FUEL (Application.offline)
Reporter | ||
Updated•17 years ago
|
Assignee: gavin.sharp → nobody
Status: ASSIGNED → NEW
Priority: P2 → --
Target Milestone: mozilla1.9alpha1 → ---
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•