Closed Bug 2065655 Opened 18 days ago Closed 11 days ago

Remove unnecessary module import in browser/components/urlbar/tests/browser/head.js

Categories

(Firefox :: Address Bar, task, P5)

task

Tracking

()

RESOLVED FIXED
157 Branch
Tracking Status
firefox157 --- fixed

People

(Reporter: scunnane, Assigned: dchen705, Mentored)

References

Details

(Keywords: good-first-bug, Whiteboard: [sng])

Attachments

(1 file)

This misspelled lazy getter in browser/components/urlbar/tests/browser/head.js is unnecessary and can be deleted. It's never read, and the tests that make use of BrowserUIUtils are getting it from the browser window.

Hi Stephanie, here's my WIP patch. Just deleted the unused line with the typo.
https://phabricator.services.mozilla.com/D321776

Flags: needinfo?(scunnane)
Assignee: scunnane → nobody
Flags: needinfo?(scunnane)
Assignee: nobody → dchen705
Attachment #9631493 - Attachment description: WIP: Bug 2065655 - Remove unnecessary module import in browser/components/urlbar/tests/browser/head.js → Bug 2065655 - Remove unnecessary module import in browser/components/urlbar/tests/browser/head.js. r?scunnane!
Status: NEW → ASSIGNED
Pushed by scunnane@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/07766883103d https://hg.mozilla.org/integration/autoland/rev/ed328e6d22dc Remove unnecessary module import in browser/components/urlbar/tests/browser/head.js. r=scunnane DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 11 days ago
Resolution: --- → FIXED
Target Milestone: --- → 157 Branch
QA Whiteboard: [search][qa-triage-done-c157/b156]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: