Assignee: mccabe → karnaze
Reassigning to Eric.
*** Bug 22705 has been marked as a duplicate of this bug. ***
Assignee: pollmann → vidur
Component: HTML Form Controls → DOM Level 0
OS: Windows 95 → All
Hardware: PC → All
Vidur, wasn't this bug fixed last millenium? (bug 15133 and bug 15185) I'm not too familiar with this area, can you take a look at this bug? If this bug is blocking anyone, one workaround is to prepend document. to the form name: <HTML> <BODY> <FORM NAME="form1"> <INPUT TYPE="button" VALUE="Shorthand" ONCLICK="alert(form1.submit)"> <INPUT TYPE="button" VALUE="Standard" ONCLICK="alert(document.form1.submit)" </FORM> </BODY> </HTML>
Checked in a temporary fix on 1/5/2000. The problem was introduced by code in nsJSContext::CompileEventHandler that unconditionally broke the scope link of compiled event handlers. While this is correct in the brutal sharing case, it shouldn't be done for HTML. Keeping the bug alive and passing it along to Brendan for a cleaner fix.
I think a flag argument to nsIScriptContext::CompileEventHandler is cleanest. Should be easy, once I update my tree from last year! /be
Cc'ing brutal sharers. This is P3 priority, but I'll fix it sooner anyway just to get it done right. /be
Crap, this is my fault. I recall talking to Waterson about HTML, but apparently I didn't do the right thing in the end. Sorry, and thanks to brendan for wiping my chin.
Thank you sir, may I have another!
Created attachment 4049 [details] [diff] [review] cleaner fix to the QI-again-for-event-handler-owner fix
Seeking vidur re-review. /be
Brendan's patch fixes the testcase on my linux build.
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
The cleaner fix vidur and I like better is now in. /be
Guys: similar bug exists on http://tvguide.netscape.com/listings The submit() code doesn't update the pull down JS box (broadcast, cable, satellite).
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Uh, this is not the bug that was once on my list, and that I closed. But since this bug got revived after death, instead of a new one being filed, I'll let jst have it (or better, make a new one). /be
Assignee: brendan → jst
Status: REOPENED → NEW
Mass moving old milestone bugs to M16
Target Milestone: M13 → M16
Looks like this is working now, I don't know what fixed it tho, marking WORKSFORME
Status: NEW → RESOLVED
Last Resolved: 19 years ago → 18 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.