Closed Bug 1772262 Opened 2 years ago Closed 1 year ago

[meta] Remove osfile.jsm from toolkit/

Categories

(Toolkit Graveyard :: OS.File, task)

Tracking

(firefox115 fixed)

RESOLVED FIXED
115 Branch
Tracking Status
firefox115 --- fixed

People

(Reporter: barret, Assigned: barret)

References

(Blocks 1 open bug)

Details

(Keywords: meta)

Attachments

(1 obsolete file)

We should port all usages of osfile.jsm to IOUtils and PathUtils

Depends on: 1725615
Depends on: 1772261
Depends on: 1772263
Blocks: 1737308
Depends on: 1772926
Depends on: 1772927
Depends on: 1772928
Depends on: 1772929
Depends on: 1772930
Depends on: 1772931
Depends on: 1772932
Depends on: 1772933
Depends on: 1772934
Depends on: 1772935
Depends on: 1772936
Depends on: 1772937
Depends on: 1772938
Depends on: 1772939
Depends on: 1772940
Depends on: 1772941
Depends on: 1772942
Depends on: 1772943
Depends on: 1772944

The Bugbug bot thinks this bug should belong to the 'Toolkit::OS.File' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: General → OS.File
Depends on: 1776305
Depends on: 1776307
Depends on: 1785318
Depends on: 1801445
Blocks: 1803179
No longer blocks: 1803179
Depends on: 1803179
Depends on: 1803182
Depends on: 1803183
Depends on: 1818868
Duplicate of this bug: 1824575

This patch removes the vast majority of OS.File and support code. A few things remain:

  • The nsIOSFileConstantsService still exists, but the path related constants
    (OS.Constants.Path.*) are no longer added to the OS object. The plan is to
    replace this with a proper service e.g. Services.osConstants or similar) in
    bug 1786885.
  • There is still support for OS.File errors in ErrorSanitizer, which will be
    removed in bug 1775167.
  • The OS.File to IOUtils migration guide will be rewritten as general IOUtils
    documentation in bug 1830097.
  • dom/base/Document.cpp has a workaround for not loading osfile.jsm at startup,
    which may want to be reconsidered in bug 1830100.

So long and thanks for all the I/O.

Assignee: nobody → brennie
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED

Comment on attachment 9330409 [details]
Bug 1772262 - Remove OS.File et al. r?gijs

Revision D176543 was moved to bug 1776480. Setting attachment 9330409 [details] to obsolete.

Attachment #9330409 - Attachment is obsolete: true
Target Milestone: --- → 115 Branch
Product: Toolkit → Toolkit Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: