Bug 28761

Summary: clipboard file access tests use "dragentered" instead of "dragenter"
Product: WebKit Reporter: noel gordon <noel.gordon>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: eric
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
use "dragenter", not "dragentered" none

noel gordon
Reported 2009-08-26 17:11:49 PDT
Silly typo I (eseidel) made. ./http/tests/security/clipboard/resources/clipboard-file-access.js:dragTarget.addEventListener("dragentered", function() { Noel said he was interested in trying this as his first WebKit patch.
Attachments
use "dragenter", not "dragentered" (10.82 KB, patch)
2009-08-26 17:41 PDT, noel gordon 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2009-08-26 17:20:22 PDT
This was one of the many casualties of the great bugzilla memory loss event of 2009. https://lists.webkit.org/pipermail/webkit-unassigned/2009-August/130367.html
noel gordon
Comment 2 2009-08-26 17:41:12 PDT
Created attachment 38652 [details] use "dragenter", not "dragentered"
Eric Seidel (no email)
Comment 3 2009-08-26 17:47:15 PDT
Comment on attachment 38652 [details] use "dragenter", not "dragentered" clicking patch checkbox.
Eric Seidel (no email)
Comment 4 2009-08-27 11:32:22 PDT
Comment on attachment 38652 [details] use "dragenter", not "dragentered" Looks OK.
Eric Seidel (no email)
Comment 5 2009-08-27 13:30:06 PDT
Comment on attachment 38652 [details] use "dragenter", not "dragentered" Clearing flags on attachment: 38652 Committed r47832: <http://trac.webkit.org/changeset/47832>
Eric Seidel (no email)
Comment 6 2009-08-27 13:30:10 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.