Closed Bug 1476135 Opened 2 years ago Closed 2 years ago
Remove unused File
Utils .jsm import from layout/tools/reftest/bootstrap .js
59 bytes, text/x-review-board-request
No description provided.
Need to remove this line only? https://searchfox.org/mozilla-central/source/layout/tools/reftest/bootstrap.js#7
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.
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 firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/23709ec5598c Remove unused FileUtils.jsm import from layout/tools/reftest/bootstrap.js r=marco
You need to log in before you can comment on or make changes to this bug.