[webkit-reviews] review granted: [Bug 227915] Port dialog close event to modern event handling code : [Attachment 433818] Patch

[bugzilla-daemon at webkit.org]

Chris Dumez <cdumez at apple.com> has granted Tim Nguyen (:ntim)
<ntim at apple.com>'s request for review:
Bug 227915: Port dialog close event to modern event handling code
https://bugs.webkit.org/show_bug.cgi?id=227915

Attachment 433818: Patch

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




--- Comment #9 from Chris Dumez <cdumez at apple.com> ---
Comment on attachment 433818
  --> https://bugs.webkit.org/attachment.cgi?id=433818
Patch

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

r=me with nit. Please make sure the bots are happy before landing.

> Source/WebCore/html/HTMLDialogElement.cpp:93
> +    document().eventLoop().queueTask(TaskSource::UserInteraction,
[protectedThis = GCReachableRef<HTMLDialogElement>(*this)] {

With C++17, you can probably write: `protectedThis = GCReachableRef { *this }`