-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WebAuthn] Clean up observers in CcidService #14274
[WebAuthn] Clean up observers in CcidService #14274
Conversation
EWS run on previous version of this PR (hash 246d7a2) |
246d7a2
to
6896738
Compare
I think the style checker error here is erroneous as it's treating the forward declaration as an identifier. |
EWS run on current version of this PR (hash 6896738) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable. Is there a way you could test this behavior works properly?
I tested manually with local logging to verify observers are added and removed as expected. |
https://bugs.webkit.org/show_bug.cgi?id=257240 rdar://109060751 Reviewed by Brent Fulgham. This change ensures removeObserver:forKeyPath: is called for each observer we add via addObserver in CcidService. * Source/WebKit/UIProcess/WebAuthentication/Cocoa/CcidService.h: * Source/WebKit/UIProcess/WebAuthentication/Cocoa/CcidService.mm: (WebKit::CcidService::~CcidService): (WebKit::CcidService::removeObservers): (WebKit::CcidService::platformStartDiscovery): (WebKit::CcidService::updateSlots): (-[_WKSmartCardSlotStateObserver observeValueForKeyPath:ofObject:change:context:]): (-[_WKSmartCardSlotStateObserver removeObserver]): Canonical link: https://commits.webkit.org/264485@main
6896738
to
52e6f05
Compare
Committed 264485@main (52e6f05): https://commits.webkit.org/264485@main Reviewed commits have been landed. Closing PR #14274 and removing active labels. |
52e6f05
6896738