RESOLVED FIXED Bug 61218
Add delegate methods about focus and blur to all elements 
https://bugs.webkit.org/show_bug.cgi?id=61218
Summary Add delegate methods about focus and blur to all elements
Enrica Casucci
Reported 2011-05-20 14:42:32 PDT
http://trac.webkit.org/changeset/45723 added these delegates for form elements. We now want to add them for all the elements. We also need to change the delegate names from formDidFocus and formDidBlur to something less specific, like elementDidFocus and elementDidBlur.
Attachments
Patch (8.07 KB, patch)
2011-05-20 15:11 PDT, Enrica Casucci 		ddkilzer: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Enrica Casucci
Comment 1 2011-05-20 14:44:22 PDT
We can afford to change these names since only the Mac implementation is using it.
Enrica Casucci
Comment 2 2011-05-20 15:11:45 PDT
Created attachment 94277 [details] Patch
David Kilzer (:ddkilzer)
Comment 3 2011-05-20 15:17:08 PDT
Comment on attachment 94277 [details] Patch r=me
Enrica Casucci
Comment 4 2011-05-20 15:21:55 PDT
<rdar://problem/9365073> and <rdar://problem/9299509>
Enrica Casucci
Comment 5 2011-05-20 15:27:31 PDT
http://trac.webkit.org/changeset/86988
mitz
Comment 6 2018-10-03 21:07:39 PDT
Comment on attachment 94277 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=94277&action=review > Source/WebKit/mac/WebView/WebUIDelegatePrivate.h:227 > +- (void)webView:(WebView *)sender elementDidFocusNode:(DOMNode *)node; > +- (void)webView:(WebView *)sender elementDidBlurNode:(DOMNode *)node; These delegate methods were renamed, but the code above in WebChromeClient.mm was never changed to call the new methods.
Note You need to log in before you can comment on or make changes to this bug.