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)
Firefox
Address Bar
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.
Updated•18 days ago
|
Hi Stephanie, here's my WIP patch. Just deleted the unused line with the typo.
https://phabricator.services.mozilla.com/D321776
Flags: needinfo?(scunnane)
| Reporter | ||
Updated•12 days ago
|
Assignee: scunnane → nobody
Flags: needinfo?(scunnane)
Updated•12 days ago
|
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
Comment 4•11 days ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 11 days ago
status-firefox157:
--- → fixed
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.
Description
•