Closed Bug 1476135 Opened 2 years ago Closed 2 years ago

Remove unused FileUtils.jsm import from layout/tools/reftest/bootstrap.js

Categories

(Testing :: Reftest, enhancement)

enhancement
Not set
normal

Tracking

(firefox63 fixed)

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: marco, Assigned: bharatraghunthan9767, Mentored)

Details

(Keywords: good-first-bug)

Attachments

(1 file)

No description provided.
Need to remove this line only?
https://searchfox.org/mozilla-central/source/layout/tools/reftest/bootstrap.js#7
Flags: needinfo?(mcastelluccio)
Comment on attachment 8993039 [details]
Bug 1476135 - Remove unused FileUtils.jsm import from layout/tools/reftest/bootstrap.js

Is that all that's required or are there more instances of FileUtils.jsm that need to be removed?
Attachment #8993039 - Flags: review?(mcastelluccio)
(In reply to Manish Kumar [:manishkk] from comment #1)
> Need to remove this line only?
> https://searchfox.org/mozilla-central/source/layout/tools/reftest/bootstrap.
> js#7

Yes, that was it. Sorry but Bharat has uploaded a patch in the meantime :)
There are a lot of other good first bugs to pick, e.g. bug 1476137.
Flags: needinfo?(mcastelluccio)
Comment on attachment 8993039 [details]
Bug 1476135 - Remove unused FileUtils.jsm import from layout/tools/reftest/bootstrap.js

https://reviewboard.mozilla.org/r/257850/#review264956

Thanks!
Attachment #8993039 - Flags: review?(mcastelluccio) → review+
Pushed by mcastelluccio@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/23709ec5598c
Remove unused FileUtils.jsm import from layout/tools/reftest/bootstrap.js r=marco
https://hg.mozilla.org/mozilla-central/rev/23709ec5598c
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Assignee: nobody → bharatraghunthan9767
You need to log in before you can comment on or make changes to this bug.