Closed
Bug 1292995
Opened 8 years ago
Closed 8 years ago
Replace in-tree consumer of non-standard Iterator() with Object.{values,entries} in services/
Categories
(Cloud Services :: General, defect)
Cloud Services
General
Tracking
(firefox51 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox51 | --- | fixed |
People
(Reporter: arai, Assigned: sumi29, Mentored)
References
Details
(Whiteboard: [good first bug] [lang=js])
Attachments
(1 file)
separated from bug 1290637. see bug 1290637 for the details. Required code changes are following: * Check each usage of non-standard Iterator [1] function, and replace it with Object.entries [2] or Object.values [3], or something appropriate for the specific usage Here's the rough list for Iterator() usage (not exhaustive) https://dxr.mozilla.org/mozilla-central/search?q=%22+Iterator(%22+path%3Aservices%2F&redirect=false for example: for (let [k, v] in Iterator(obj)) { ... } can be replaced to: for (let [k, v] of Object.entries(obj)) { ... } another example: for (let [, v] in Iterator(array)) { ... } can be replaced to: for (let v of array) { ... } [1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Iterator [2] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/entries [3] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/values
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•8 years ago
|
||
All tests in the services/ folder passed, so hopefully this changeset won't have the same mistakes as the one for bug 1290698.
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → sumi29
Status: NEW → ASSIGNED
Reporter | ||
Comment 3•8 years ago
|
||
mozreview-review |
Comment on attachment 8779154 [details] Bug 1292995 - Replace Iterator() with Object.{values,entries} in services/; https://reviewboard.mozilla.org/r/70186/#review67512 Thank you for taking this :) Looks good. forwarding to gps, as I'm not module owner/peer here.
Reporter | ||
Updated•8 years ago
|
Attachment #8779154 -
Flags: review?(gps)
Reporter | ||
Updated•8 years ago
|
Attachment #8779154 -
Flags: review?(arai.unmht) → feedback+
Comment 4•8 years ago
|
||
mozreview-review |
Comment on attachment 8779154 [details] Bug 1292995 - Replace Iterator() with Object.{values,entries} in services/; https://reviewboard.mozilla.org/r/70186/#review67700
Attachment #8779154 -
Flags: review?(gps) → review+
Reporter | ||
Comment 5•8 years ago
|
||
Thanks! here's try run https://treeherder.mozilla.org/#/jobs?repo=try&revision=e44fe871b9b1
Pushed by arai_a@mac.com: https://hg.mozilla.org/integration/autoland/rev/7bab284d7910 Replace Iterator() with Object.{values,entries} in services/; r=gps
Comment 7•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7bab284d7910
You need to log in
before you can comment on or make changes to this bug.
Description
•