Make sure that nsFormAutoComplete modifications from bug 1294502 will work in other non-Firefox XUL apps.

RESOLVED FIXED

Status

()

Toolkit
Form Manager
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mconley, Unassigned)

Tracking

unspecified
Points:
---

Firefox Tracking Flags

(firefox50 unaffected, firefox51 fix-optional, firefox52 fix-optional, firefox53 fix-optional)

Details

I'm not too worried about Fennec - I don't think it'll be affected. I'm more worried about SeaMonkey and Thunderbird. I suspect bug 1294502 may have broken form autocomplete popups for them.

I'm happy to land additional patches to fix these potential consumers of form autocompletion.
I'm not 100% sure where <xul:browser>'s might use the form autocomplete stuff in Thunderbird or SeaMonkey. Magnus, do you know where Thunderbird might use it?

RSX11M, do you know where SeaMonkey might use it?

If there's breakage, the first part of the solution is to make sure there's a panel like this per window: http://searchfox.org/mozilla-central/rev/ae78ab94fadabc89fc6258d03c4a1a70f763f43a/browser/base/content/browser.xul#144 and that <xul:browser>'s that want to use form autocomplete have an autocompletepopup attribute set to the ID of the PopupAutoComplete panel.

That might not be 100% enough though - there might need to be some work for PopupAutoComplete to make it app-agnostic, as I believe there are some gBrowser-isms in there that I'm not sure the other apps have.

Anyhow, let me know if there's breakage.
Flags: needinfo?(rsx11m.pub)
Flags: needinfo?(mkmelin+mozilla)

Comment 2

2 years ago
For form items, SeaMonkey is definitely using the Toolkit implementation.
The URL bar may be separate implementation.

Forwarding the NI? request to IanN and Ratty.
Flags: needinfo?(rsx11m.pub)
Flags: needinfo?(philip.chee)
Flags: needinfo?(iann_bugzilla)
Seamonkey sure is broken now. It was reported in bug 1300547. I tried to fix it there and it seems to be semi working. The resulting list box which contains the results is always empty but seems to be the correcnt size. Bug 1298204 is about a similar issue but the blank lines can not be selected in Seamonkey.
Just tried the fix for bug 1298204 in suite and it does not help with the patch in bug 1300547

Comment 5

2 years ago
As far as I know, form auto-completion isn't really used within Thunderbird. Possibly in some extension like ThunderBrowse then...
Flags: needinfo?(mkmelin+mozilla)
Mike,

could you take a look at bug 1300547. This is half working for SeaMonkey but I have no clue what might be missing. Do you know if Firefox does the form fill itself and doesn't use toolkit?

FRG
Flags: needinfo?(mconley)
Yep, looked, and commented. Thanks for working on that.
Flags: needinfo?(mconley)
status-firefox50: --- → unaffected
status-firefox51: --- → fix-optional
status-firefox52: --- → fix-optional
status-firefox53: --- → fix-optional
Matt, Thanks. Thought this bug was already closed. I think you can close it. SeaMonkey 2.48 and up were fixed in bug 1300547. TB and IM do not seem to be affected.
Flags: needinfo?(philip.chee)
Flags: needinfo?(iann_bugzilla)
See Also: → bug 1300547

Comment 9

2 years ago
Mike, can we close this?
Flags: needinfo?(mconley)
Yeah, I think so.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Flags: needinfo?(mconley)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.