1573836 - Make autocomplete fillin work in out of process iframes

Status: RESOLVED FIXED

(Toolkit :: Autocomplete, task, P3)

Description

[Neil Deakin]

Some login manager tests rely on this. This involves changing toolkit/modules/AutoCompletePopupContent.jsm (child side) and toolkit/components/satchel/AutoCompletePopup.jsm (parent side) as well as the event listeners in toolkit/content/browser-content.js

Comment 1

[Neil Deakin]

Attachment: Bug 1573836, make autocomplete component fission compatible, r=mak,MattN

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Hey MattN / mak,

Do you think you'll have some cycles to give Enn some feedback here soon?

Comment 3

[Marco Bonardo [:mak]]

In italian we use to say "ni", for "no e si" :) It's a complicate patch at the wrong time (we are temporarily missing 2 developers in the Search team and we have 5 projects ongoing), and I don't have all the knowledge necessary to go through it by memory. I started looking at it, and I got lost.
I'll do a first pass by Monday, sorry for delays.

Comment 4

[Matthew N. [:MattN]]

I was away last week but will try look at it tomorrow.

Comment 5

[Matthew N. [:MattN]]

I reviewed this yesterday.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/25ba54160c0a
make autocomplete component fission compatible, r=mak,MattN

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/25ba54160c0a