Bug 50477

Summary: REGRESSION(r71934): input event fires twice when editing text inside a text input.
Product: WebKit Reporter: Dimitri Glazkov (Google) <dglazkov>
Component: New BugsAssignee: Dimitri Glazkov (Google) <dglazkov>
Status: RESOLVED FIXED    
Severity: Normal CC: darin
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 46015    
Attachments:
Description Flags
Patch
none
Now with a better ChangeLog darin: review+

Description Dimitri Glazkov (Google) 2010-12-03 13:07:12 PST
REGRESSION(r71934): input event fires twice when editing text inside a text input.
Comment 1 Dimitri Glazkov (Google) 2010-12-03 13:09:52 PST
Created attachment 75532 [details]
Patch
Comment 2 Dimitri Glazkov (Google) 2010-12-03 13:10:32 PST
Comment on attachment 75532 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=75532&action=review

> LayoutTests/fast/forms/text-input-event.html:44
>  \ No newline at end of file

Ha. I forgot the newline again. I'll add upon landing.
Comment 3 Dimitri Glazkov (Google) 2010-12-03 13:13:21 PST
Created attachment 75533 [details]
Now with a better ChangeLog
Comment 4 Darin Adler 2010-12-03 13:25:29 PST
Comment on attachment 75533 [details]
Now with a better ChangeLog

I’m not sure I completely understand our approach; seems more like we would not want the event to exit the shadow.

This changes the timing of the input event too. It now happens earlier. Does that have any detectable side effects?
Comment 5 Dimitri Glazkov (Google) 2010-12-03 14:06:11 PST
(In reply to comment #4)
> (From update of attachment 75533 [details])
> I’m not sure I completely understand our approach; seems more like we would not want the event to exit the shadow.
> 
> This changes the timing of the input event too. It now happens earlier. Does that have any detectable side effects?

It actually happens later now. InputElement::setValueFromRenderer is invoked from HTMLFormControlElementWithState::defaultEventHandler.

However, both of these events are a response to the same specific webkitEditableContentChangedEvent, so there's really no perceptible difference in when they occur.
Comment 6 Dimitri Glazkov (Google) 2010-12-03 16:12:00 PST
Committed r73319: <http://trac.webkit.org/changeset/73319>
Comment 7 Dimitri Glazkov (Google) 2011-03-14 10:02:40 PDT
*** Bug 50476 has been marked as a duplicate of this bug. ***