Closed
Bug 1671274
Opened 4 years ago
Closed 4 years ago
SearchSettings.jsm and SearchService.jsm import OS.File during startup
Categories
(Toolkit Graveyard :: OS.File, task)
Toolkit Graveyard
OS.File
Tracking
(firefox89 fixed)
RESOLVED
FIXED
89 Branch
Tracking | Status | |
---|---|---|
firefox89 | --- | fixed |
People
(Reporter: emmamalysz, Assigned: masterwayz, Mentored)
References
Details
(Whiteboard: [lang=js])
Attachments
(1 file)
We should remove the instances of OS.File from SearchSettings.jsm and use their IOUtils equivalents: https://searchfox.org/mozilla-central/rev/9c72508fcf2bba709a5b5b9eae9da35e0c707baa/toolkit/components/search/SearchSettings.jsm#13,99-100,103,215,217
Assignee | ||
Updated•4 years ago
|
Assignee: nobody → michael
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•4 years ago
|
||
Reporter | ||
Comment 3•4 years ago
|
||
I spoke wiith :masterwayz, and we are going to cover both removals in this bug now.
Depends on: 1649601
Summary: SearchSettings.jsm imports OS.File during startup → SearchSettings.jsm and SearchService.jsm import OS.File during startup
Updated•4 years ago
|
Attachment #9188122 -
Attachment description: Bug 1671274 - Convert SearchSettings.jsm to use IOUtils instead of OS.File r=emalysz → WIP: Bug 1671274 - Convert SearchSettings.jsm and SearchService.jsm to no longer use OS.File r=emalysz
Updated•4 years ago
|
Attachment #9188122 -
Attachment description: WIP: Bug 1671274 - Convert SearchSettings.jsm and SearchService.jsm to no longer use OS.File r=emalysz → Bug 1671274 - Convert SearchSettings.jsm and SearchService.jsm to no longer use OS.File r=emalysz
Pushed by michelle@masterwayz.nl:
https://hg.mozilla.org/integration/autoland/rev/6b0908a8b7de
Convert SearchSettings.jsm and SearchService.jsm to no longer use OS.File r=barret
Comment 5•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox89:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
Updated•2 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•