[webkit-reviews] review granted: [Bug 234047] Remove [DocumentEventHandler] extended attribute : [Attachment 446465] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Alexey Shvayka <ashvayka at apple.com>'s
request for review:
Bug 234047: Remove [DocumentEventHandler] extended attribute
https://bugs.webkit.org/show_bug.cgi?id=234047

Attachment 446465: Patch

https://bugs.webkit.org/attachment.cgi?id=446465&action=review




--- Comment #2 from Darin Adler <darin at apple.com> ---
Comment on attachment 446465
  --> https://bugs.webkit.org/attachment.cgi?id=446465
Patch

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

No progression on WPT tests?

> Source/WebCore/ChangeLog:3
> +	   Remove [DocumentEventHandler] extended attribute

In future, consider making the patch that removes the last use separate from
the patch removing the implementation.

Makes it a little easier to concentrate on those two separate concerns, and I
don’t think there are a lot of downsides.