Summary: | [watchOS] Notification listener is never unregistered | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Per Arne Vollan <pvollan> | ||||||
Component: | WebKit Misc. | Assignee: | Per Arne Vollan <pvollan> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bfulgham, commit-queue, mjs, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Per Arne Vollan
2020-02-10 07:16:21 PST
Created attachment 390250 [details]
Patch
Comment on attachment 390250 [details]
Patch
Maybe worth making an API test in case they are run there some day, or in case we make this mistake on another platform?
Code change looks fine but I suggest adding a test.
(In reply to Maciej Stachowiak from comment #3) > Comment on attachment 390250 [details] > Patch > > Maybe worth making an API test in case they are run there some day, or in > case we make this mistake on another platform? > > Code change looks fine but I suggest adding a test. Thanks for reviewing! I will look into adding a test. Created attachment 390297 [details]
Patch
Comment on attachment 390297 [details] Patch Clearing flags on attachment: 390297 Committed r256225: <https://trac.webkit.org/changeset/256225> |