Closed Bug 1573836 Opened 10 months ago Closed 8 months ago

Make autocomplete fillin work in out of process iframes

Categories

(Toolkit :: Autocomplete, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla71
Fission Milestone M4
Tracking Status
firefox71 --- fixed

People

(Reporter: enndeakin, Assigned: enndeakin)

References

(Blocks 1 open bug, Regressed 1 open bug)

Details

Attachments

(1 file)

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

Priority: -- → P3
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Blocks: 1582722

Hey MattN / mak,

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

Flags: needinfo?(mak77)
Flags: needinfo?(MattN+bmo)

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.

Flags: needinfo?(mak77)

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

I reviewed this yesterday.

Flags: needinfo?(MattN+bmo)
Pushed by neil@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/25ba54160c0a
make autocomplete component fission compatible, r=mak,MattN
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Blocks: 1548575
Depends on: 1595875
Fission Milestone: --- → M4
Regressions: 1604336
Regressions: 1610539
Regressions: 1611900
You need to log in before you can comment on or make changes to this bug.