Summary: | Make it possible to cancel an EventLoopTask and use in HTMLDetailsElement | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||||||||
Component: | DOM | Assignee: | Ryosuke Niwa <rniwa> | ||||||||||
Status: | RESOLVED LATER | ||||||||||||
Severity: | Normal | CC: | cdumez, changseok, esprehn+autocc, ews-watchlist, gyuyoung.kim, kangil.han, ntim, simon.fraser, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=228863 | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 202843 | ||||||||||||
Attachments: |
|
Description
Ryosuke Niwa
2021-07-28 22:02:57 PDT
Created attachment 434494 [details]
Patch
Created attachment 434496 [details]
Patch
Created attachment 434504 [details]
Patch
Created attachment 434509 [details]
Patch
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<> ? (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. Does bug 228863 somewhat solve this? (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. |