RESOLVED LATER 228577
Make it possible to cancel an EventLoopTask and use in HTMLDetailsElement 
https://bugs.webkit.org/show_bug.cgi?id=228577
Summary Make it possible to cancel an EventLoopTask and use in HTMLDetailsElement
Ryosuke Niwa
Reported 2021-07-28 22:02:57 PDT
Add an identifier for each task so that we can use it to cancel the task later. Migrate HTMLDetailsElement's toggle event to the event loop using that.
Attachments
Patch (12.14 KB, patch)
2021-07-28 22:18 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Patch (12.22 KB, patch)
2021-07-28 22:52 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Patch (12.42 KB, patch)
2021-07-29 02:04 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Patch (12.42 KB, patch)
2021-07-29 03:15 PDT, Ryosuke Niwa 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2021-07-28 22:18:44 PDT
Created attachment 434494 [details] Patch
Ryosuke Niwa
Comment 2 2021-07-28 22:52:55 PDT
Created attachment 434496 [details] Patch
Ryosuke Niwa
Comment 3 2021-07-29 02:04:12 PDT
Created attachment 434504 [details] Patch
Ryosuke Niwa
Comment 4 2021-07-29 03:15:02 PDT
Created attachment 434509 [details] Patch
Simon Fraser (smfr)
Comment 5 2021-07-29 09:47:25 PDT
Comment on attachment 434509 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=434509&action=review > Source/WebCore/dom/EventLoop.h:76 > + enum class TaskIdentifier : uint64_t { Invalid = 0 }; Is there a reason not to use ObjectIdentifier<> ?
Ryosuke Niwa
Comment 6 2021-07-29 14:49:56 PDT
(In reply to Simon Fraser (smfr) from comment #5) > Comment on attachment 434509 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=434509&action=review > > > Source/WebCore/dom/EventLoop.h:76 > > + enum class TaskIdentifier : uint64_t { Invalid = 0 }; > > Is there a reason not to use ObjectIdentifier<> ? ObjectIdentifier will increase the size of EventLoopTask by 8 bytes. I'm avoiding that by using the combination of the pointer value & 16-bit salting.
Radar WebKit Bug Importer
Comment 7 2021-08-04 22:03:18 PDT
<rdar://problem/81549670>
Tim Nguyen (:ntim)
Comment 8 2022-01-19 05:19:52 PST
Does bug 228863 somewhat solve this?
Ryosuke Niwa
Comment 9 2022-01-19 21:21:06 PST
(In reply to Tim Nguyen (:ntim) from comment #8) > Does bug 228863 somewhat solve this? It doesn't since the main point of this patch is to add the ability to cancel a task in the event loop but Chris sort of fixed that by adding CancelableTask in https://trac.webkit.org/r278543. Let's defer this to later.
Note You need to log in before you can comment on or make changes to this bug.