Closed
Bug 1695309
Opened 4 years ago
Closed 2 years ago
Replace CommonUtils.{read,write}JSON usage with IOUtils.{read,write}JSON
Categories
(Toolkit Graveyard :: OS.File, enhancement, P3)
Toolkit Graveyard
OS.File
Tracking
(firefox108 fixed)
RESOLVED
FIXED
108 Branch
| Tracking | Status | |
|---|---|---|
| firefox108 | --- | fixed |
People
(Reporter: barret, Assigned: barret)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
IOUtils.{read,write}JSON does all the IO and de/serialization work without leaving C++. Some JSMs import CommonUtils only for {read,write}JSON so we can remove imports in those cases too
|
Assignee | |
Comment 2•2 years ago
|
||
Pushed by brennie@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bc3f9439cf22
Replace CommonUtils.{read,write}JSON with IOUtils.{read,write}JSON r=Gijs,markh
|
||
Comment 4•2 years ago
|
||
Backed out for causing multiple xpcshell failures.
Failure log for when the job fails on test_device.js
Failure log for when the job fails on test_ProfileAge.js
Failure log for when the job fails on test_client_id.js
Failure log for TV job that first fails on test_TelemetryController.js
Failure log for TV job that first fails on test_TelemetryEvents_buildFaster.js
Failure log for TV job that first fails on test_TelemetrySession.js
Failure log for TV job that first fails on test_loginmgr_storage.js
Flags: needinfo?(brennie)
|
|
Assignee | |
Updated•2 years ago
|
Flags: needinfo?(brennie)
Pushed by brennie@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ac2210755eee
Replace CommonUtils.{read,write}JSON with IOUtils.{read,write}JSON r=Gijs,markh
|
||
Comment 6•2 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox108:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 108 Branch
|
||
Updated•1 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•