Bug 36266

Summary: Fire Focus Event type: focusin events
Product: WebKit Reporter: Andy Estes <aestes>
Component: WebCore JavaScriptAssignee: Dave Hyatt <hyatt>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, simon.fraser, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Macintosh   
OS: OS X 10.4   
Attachments:
Description Flags
Patch
none
Patch sam: review+

Description Andy Estes 2010-03-17 19:51:21 PDT
2010-03-17 17:46:22 Andy Estes:
Safari fails the following test: http://samples.msdn.microsoft.com/ietestcenter/domevents/domevents_harness.htm?url=./focusin.html.

Safari does not fire focus events when registered for using the name 'focusin'.  This is a valid event name as defined by the W3C DOM Level 3 Events Working Draft at http://www.w3.org/TR/DOM-Level-3-Events/.  Safari does implement the 'DOMFocusIn' event, which appears to serve the same purpose.

<rdar://problem/7765952>
Comment 1 Dave Hyatt 2010-03-19 11:49:09 PDT
Created attachment 51176 [details]
Patch
Comment 2 Simon Fraser (smfr) 2010-03-19 11:51:56 PDT
Comment on attachment 51176 [details]
Patch

r=me
Comment 3 Dave Hyatt 2010-03-19 12:42:47 PDT
Comment on attachment 51176 [details]
Patch

Clearing flag, since we need to support the old event names still.  New patch is going to be much more complicated. :)
Comment 4 Dave Hyatt 2010-03-22 12:28:55 PDT
Created attachment 51329 [details]
Patch
Comment 5 WebKit Review Bot 2010-03-22 12:32:29 PDT
Attachment 51329 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/dom/Node.cpp:2726:  Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side.  [whitespace/operators] [4]
WebCore/dom/Event.cpp:269:  Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side.  [whitespace/operators] [4]
Total errors found: 2 in 17 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 Dave Hyatt 2010-03-23 10:53:19 PDT
Fixed in r56402.