Bug 61218 - Add delegate methods about focus and blur to all elements
Summary: Add delegate methods about focus and blur to all elements
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.6
: P2 Normal
Assignee: Enrica Casucci
Keywords: InRadar, PlatformOnly
Depends on:
Reported: 2011-05-20 14:42 PDT by Enrica Casucci
Modified: 2018-10-03 21:07 PDT (History)
2 users (show)

See Also:

Patch (8.07 KB, patch)
2011-05-20 15:11 PDT, Enrica Casucci
ddkilzer: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Enrica Casucci 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.
Comment 1 Enrica Casucci 2011-05-20 14:44:22 PDT
We can afford to change these names since only the Mac implementation is using it.
Comment 2 Enrica Casucci 2011-05-20 15:11:45 PDT
Created attachment 94277 [details]
Comment 3 David Kilzer (:ddkilzer) 2011-05-20 15:17:08 PDT
Comment on attachment 94277 [details]

Comment 4 Enrica Casucci 2011-05-20 15:21:55 PDT
<rdar://problem/9365073> and <rdar://problem/9299509>
Comment 5 Enrica Casucci 2011-05-20 15:27:31 PDT
Comment 6 mitz 2018-10-03 21:07:39 PDT
Comment on attachment 94277 [details]

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.