Closed Bug 769310 Opened 12 years ago Closed 12 years ago

[OS.File] Separate OS.Path from worker-only code

Categories

(Core :: Networking: File, enhancement)

Product:
Core
Component:
Networking: File
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla17

People

(Reporter: Yoric, Assigned: Yoric)

References

Details

Attachments

(2 files, 4 obsolete files)

Windows back-end
(deleted), patch
Yoric
: review+
Details | Diff | Splinter Review
Unix back-end
(deleted), patch
Yoric
: review+
Details | Diff | Splinter Review
We will want to execute OS.Path on the main thread. For the moment, however, OS.Path is locked to the worker code. This should change.
Attached patch Unix back-end (obsolete) (deleted) — — Splinter Review
Assignee: nobody → dteller
Attached patch Windows back-end (obsolete) (deleted) — — Splinter Review
Whiteboard: [autoland-inbound]
Attached patch Windows back-end (obsolete) (deleted) — — Splinter Review
Basically a trivial patch: OS.Path moves to its own file.
Attachment #639061 - Attachment is obsolete: true
Attachment #641865 - Flags: review?(taras.mozilla)
Whiteboard: [autoland-inbound]
Attached patch Unix back-end (obsolete) (deleted) — — Splinter Review
Attachment #639060 - Attachment is obsolete: true
Attachment #641867 - Flags: review?(taras.mozilla)
Comment on attachment 641865 [details] [diff] [review] Windows back-end rubberstamp
Attachment #641865 - Flags: review?(taras.mozilla) → review+
Comment on attachment 641867 [details] [diff] [review] Unix back-end rubberstamp
Attachment #641867 - Flags: review?(taras.mozilla) → review+
Attached patch Windows back-end (deleted) — — Splinter Review
Attachment #641865 - Attachment is obsolete: true
Attachment #643461 - Flags: review+
Attached patch Unix back-end (deleted) — — Splinter Review
Attachment #641867 - Attachment is obsolete: true
Attachment #643462 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/6ea3217e0176 https://hg.mozilla.org/mozilla-central/rev/c91e4d6df9a4
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: