Open Bug 690570 Opened 9 years ago Updated 2 years ago
e10s add-on support for ns
IForm Submit Observer
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.
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 → ---
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.
Summary: e10s support for nsIFormSubmitObserver → e10s add-on support for nsIFormSubmitObserver
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.