1483738 - Lazily load FormSubmitObserver code

Status: RESOLVED FIXED

(Firefox :: General, enhancement)

Description

[Kris Maglione [:kmag]]

This requires making the invalidformsubmit notifications... marginally more sane. They currently register things with the observer service which are not actually observers, and then having some other code iterate over registered observers and muck with them.

For my purposes, I'd rather deal with a DOM event, but I'm also going to change the signature of these observers to be marginally more sane so that I have an easily accessible nsTArray to work with and continue to dispatch the "observer" callbacks as well, so I don't have to change Android code just now.

Comment 1

[Kris Maglione [:kmag]]

Attachment: Bug 1483738: Part 1 - Change notifyInvalidSubmit to take a plain array. r=mconley

This makes things saner for all consumers, which makes it worth doing on its
own. But it also gives me an easy nsTArray to work with, which I can use to
dispatch DOM events with array properties.

Comment 2

[Kris Maglione [:kmag]]

Attachment: Bug 1483738: Part 2 - Add MozInvalidForm event, which can be handled in place of notifyInvalidSubmit observers. r=smaug,mconley

Comment 3

[Kris Maglione [:kmag]]

Attachment: Bug 1483738: Part 3 - Use lazy MozInvalidForm listener on desktop. r=mconley

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 9001471 [details]
Bug 1483738: Part 2 - Add MozInvalidForm event, which can be handled in place of notifyInvalidSubmit observers. r=smaug,mconley

Olli Pettay [:smaug] has approved the revision.

Comment 5

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

Comment on attachment 9001472 [details]
Bug 1483738: Part 3 - Use lazy MozInvalidForm listener on desktop. r=mconley

Mike Conley (:mconley) (:⚙️) has approved the revision.

Comment 6

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

Comment on attachment 9001471 [details]
Bug 1483738: Part 2 - Add MozInvalidForm event, which can be handled in place of notifyInvalidSubmit observers. r=smaug,mconley

Mike Conley (:mconley) (:⚙️) has approved the revision.

Comment 7

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6f8bb8f44f2b612c964e1116282917e33168f989
Bug 1483738: Part 1 - Change notifyInvalidSubmit to take a plain array. r=mconley

https://hg.mozilla.org/integration/mozilla-inbound/rev/89c81ab0254376e6a7ae4a61fa2607d4ecd7b08b
Bug 1483738: Part 2 - Add MozInvalidForm event, which can be handled in place of notifyInvalidSubmit observers. r=smaug,mconley

https://hg.mozilla.org/integration/mozilla-inbound/rev/9b5aefe66457f3bbb6403dbe7c149993c7c7eb70
Bug 1483738: Part 3 - Use lazy MozInvalidForm listener on desktop. r=mconley

Comment 8

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/6f8bb8f44f2b
https://hg.mozilla.org/mozilla-central/rev/89c81ab02543
https://hg.mozilla.org/mozilla-central/rev/9b5aefe66457