Bug 55755 - Remove onformchange and onforminput events/event handlers
Summary: Remove onformchange and onforminput events/event handlers
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Alexey Proskuryakov
URL:
Keywords: InRadar
Depends on:
Blocks: HTML5Forms 26141
  Show dependency tree / graph
 
Reported: 2011-03-04 02:04 PST by Olli Pettay (:smaug)
Modified: 2011-04-05 17:34 PDT (History)
12 users (show)

See Also:

Attachments
proposed patch (37.54 KB, patch)
2011-04-04 15:08 PDT, Alexey Proskuryakov 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Alexey Proskuryakov 2011-03-04 10:43:04 PST 
<rdar://problem/9087969>
Comment 2 Alexey Proskuryakov 2011-04-04 15:08:37 PDT 
Created attachment 88140 [details]
proposed patch
Comment 3 Darin Adler 2011-04-04 15:10:37 PDT 
Comment on attachment 88140 [details]
proposed patch

How long have these been in? Have they been in Chrome and Safari releases? Mac OS X WebKit?
Comment 4 Alexey Proskuryakov 2011-04-04 15:13:25 PDT 
The code was added in bug 26141 in January 2011.
Comment 5 Kent Tamura 2011-04-04 18:44:48 PDT 
(In reply to comment #3)
> (From update of attachment 88140 [details])
> How long have these been in? Have they been in Chrome and Safari releases? Mac OS X WebKit?

Safari 5 or prior doesn't have them, and only Chrome 10 and 11 have them. If the patch is committed, Chrome 12 won't have them.
Comment 6 WebKit Commit Bot 2011-04-05 04:45:50 PDT 
Comment on attachment 88140 [details]
proposed patch

Clearing flags on attachment: 88140

Committed r82925: <http://trac.webkit.org/changeset/82925>
Comment 7 WebKit Commit Bot 2011-04-05 04:45:57 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 8 WebKit Review Bot 2011-04-05 05:29:50 PDT 
http://trac.webkit.org/changeset/82925 might have broken Qt Linux Release
The following tests are not passing:
fast/dom/Window/window-properties.html
fast/dom/Window/window-property-descriptors.html
Comment 9 Alexey Proskuryakov 2011-04-05 08:48:18 PDT 
Qt results were updated in r82941.
Comment 10 Alexey Proskuryakov 2011-04-05 17:34:38 PDT 
Windows results also need to be updated, bug 57881.

Trivial follow-up in bug 57908.