Closed Bug 532143 Opened 11 years ago Closed 11 years ago
Util .new URI should take a string or ns IFile
No description provided.
Not sold on the the first parameter's name, but I could not come up with anything better. Open to suggestions.
Comment on attachment 415490 [details] [diff] [review] v1.0 Why not just leave it called aSpec?
Attachment #415490 - Flags: review?(bzbarsky) → review+
(In reply to comment #2) > (From update of attachment 415490 [details] [diff] [review]) > Why not just leave it called aSpec? It didn't seem quite right, but I can certainly change it back. I do not have a strong opinion on this.
Whiteboard: [needs review bz][needs sr vlad] → [needs sr vlad]
How about aTarget?
(In reply to comment #4) > How about aTarget? Sold.
Fixed locally. Will attach a new patch once vlad does his sr.
Attachment #415490 - Flags: superreview?(vladimir) → superreview+
For checkin (with commit message)
Attachment #415490 - Attachment is obsolete: true
Target Milestone: --- → mozilla1.9.3a1
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [can land]
You need to log in before you can comment on or make changes to this bug.