Open Bug 690570 Opened 9 years ago Updated 2 years ago

e10s add-on support for nsIFormSubmitObserver

Categories

(Core :: DOM: Core & HTML, defect, P5)

defect

Tracking

()

REOPENED
Tracking Status
e10s + ---

People

(Reporter: Gavin, Unassigned)

References

Details

The form element validation code currently works by looking for nsIFormSubmitObservers registered with the observer service, and and lets them deal with displaying the UI for the form submission (a xul:panel anchored to the element in Firefox's case). We need to find some other way to implement that's e10s-friendly.
Whiteboard: [e10s]
No longer blocks: e10s-m1
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
Reopening this. My initial look into this centered on the browser's use of the 'invalidformsubmit' observer. However, I did some searches across the addon database and found that addons are using some of these events, specifically:

earlyformsubmit - about 25 uses
formsubmit - 0 uses
firstformsubmit - 0 uses
invalidformsubmit - 1 use
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Assignee: nobody → jmathies
http://mxr.mozilla.org/mozilla-central/source/content/html/content/src/HTMLFormElement.cpp#913

Looks like 'earlyformsubmit', 'firstformsubmit', and 'formsubmit' can be sent over via an e10s call and fired in the parent, they both carry a window pointer, action url string, and a cancel flag.

'invalidformsubmit' however carries with it a list of elements, not sure what to do about those.
Per a discussion in our weekly meeting, we plan to land the front end fix for the browser, and address addon usage of this api through the addon compat layer we're working on.
Blocks: e10s-addons
No longer blocks: 691601
Summary: e10s support for nsIFormSubmitObserver → e10s add-on support for nsIFormSubmitObserver
Assignee: jmathies → nobody
addon compat, sounds like my department these days.
Assignee: nobody → ally
Blocks: e10s-shims
No longer blocks: e10s-addons
Whiteboard: [e10s]
Assignee: ally → nobody
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven't been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.