Closed
Bug 1337480
Opened 7 years ago
Closed 7 years ago
test_417228-other-roots.js - excludedItemsFromRestore / excludedItems are undefined
Categories
(Toolkit :: Places, defect)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
In trying to turn on the no-undef rule for eslint, I've come across test_417228-other-roots.js which has some undefined variables: https://dxr.mozilla.org/mozilla-central/rev/af8a2573d0f1e9cc6f2ba0ab67d7a702a197f177/toolkit/components/places/tests/bookmarks/test_417228-other-roots.js#138 ``` if (aTest.excludedItemsFromRestore) excludedItemsFromRestore = excludedItems.concat(aTest.excludedItemsFromRestore); ``` `excludedItemsFromRestore` used to be defined as an array. However, it wasn't used, and `excludedItems` is also undefined. This seems to have been the case since the test was originally landed. I don't have enough knowledge here to know if this is something that needs fixing, or if we can just remove the code that has the undefined variables. Marco - any ideas?
Flags: needinfo?(mak77)
Comment 1•7 years ago
|
||
You can remove excludeItemsFromRestore, excludedItemsFromRestore and excludedItems.
Flags: needinfo?(mak77)
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → standard8
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8835121 [details] Bug 1337480 - Remove undefined variables in test_417228-other-roots.js as they aren't needed. https://reviewboard.mozilla.org/r/110814/#review112296
Attachment #8835121 -
Flags: review?(mak77) → review+
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/82cdda27190e Remove undefined variables in test_417228-other-roots.js as they aren't needed. r=mak
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/82cdda27190e
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•