The default bug view has changed. See this FAQ.

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

RESOLVED FIXED in mozilla17

Status

()

Core
Networking: File
--
enhancement
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Yoric, Assigned: Yoric)

Tracking

unspecified
mozilla17
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 4 obsolete attachments)

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.
Blocks: 729057
Depends on: 707696
Created attachment 639060 [details] [diff] [review]
Unix back-end
Assignee: nobody → dteller
Created attachment 639061 [details] [diff] [review]
Windows back-end
Whiteboard: [autoland-inbound]
Created attachment 641865 [details] [diff] [review]
Windows back-end

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]
Created attachment 641867 [details] [diff] [review]
Unix back-end
Attachment #639060 - Attachment is obsolete: true
Attachment #641867 - Flags: review?(taras.mozilla)

Comment 5

5 years ago
Comment on attachment 641865 [details] [diff] [review]
Windows back-end

rubberstamp
Attachment #641865 - Flags: review?(taras.mozilla) → review+

Comment 6

5 years ago
Comment on attachment 641867 [details] [diff] [review]
Unix back-end

rubberstamp
Attachment #641867 - Flags: review?(taras.mozilla) → review+
Keywords: checkin-needed
Keywords: checkin-needed
Created attachment 643461 [details] [diff] [review]
Windows back-end
Attachment #641865 - Attachment is obsolete: true
Attachment #643461 - Flags: review+
Created attachment 643462 [details] [diff] [review]
Unix back-end
Attachment #641867 - Attachment is obsolete: true
Attachment #643462 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/6ea3217e0176
https://hg.mozilla.org/integration/mozilla-inbound/rev/c91e4d6df9a4
Flags: in-testsuite-
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/6ea3217e0176
https://hg.mozilla.org/mozilla-central/rev/c91e4d6df9a4
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.