1887680 - Address feedback from https://phabricator.services.mozilla.com/D202678

Status: RESOLVED FIXED

(Toolkit :: Form Autofill, task, P2)

Comment 1

[Sergey Galich [:serg]]

Attachment: WIP: Bug 1887680 - refactoring BrowserWindowTracker use

Comment 2

[Sergey Galich [:serg]]

Attachment: Bug 1887680 - simplifying localisation

Comment 3

[Sergey Galich [:serg]]

Attachment: Bug 1887680 - linting with 100 chars break point r=#credential-management-reviewers

Depends on D205921

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1c9048aedc33
simplifying localisation r=credential-management-reviewers,mtigley,issammani

Comment 5

[Aron Cseh]

https://hg.mozilla.org/mozilla-central/rev/1c9048aedc33

Comment 6

[Dimi Lee [:dimi]]

Hi Micah, I'm not sure what needed to be done for this bug, is this bug still valid, if yes, could you share a bit of more information? thank you!

Comment 7

[Micah [:mtigley] (she/her)]

Hey Dimi! Apologies for the delay here. This bug required for us to establish testing for Megalist and moving the code from toolkit/ to browser/.

For tests, we have already begun establishing them in recent patches. But I will create a separate ticket for addressing moving the code. Let's close this ticket in favour of that work.