Bug 149384

Summary: [Mac, iOS] AccessibilityController doesn't uninstall global notification handler
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: Tools / TestsAssignee: Alexey Proskuryakov <ap>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, apinheiro, cfleizach, commit-queue, dmazzoni, jcraig, jdiggs, mario, samuel_white
Priority: P2    
Version: Safari 9   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=149217
Attachments:
Description Flags
proposed fix none

Alexey Proskuryakov
Reported 2015-09-20 00:36:07 PDT
AccessibilityController::platformResetToConsistentState() clears the pointer, but the notification handle remains set. This is observable, as accessibility/mac/loaded-notification.html changes the behavior of subsequent accessibility tests.
Attachments
proposed fix (4.96 KB, patch)
2015-09-20 00:41 PDT, Alexey Proskuryakov 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2015-09-20 00:41:07 PDT
Created attachment 261605 [details] proposed fix
WebKit Commit Bot
Comment 2 2015-09-20 08:30:27 PDT
Comment on attachment 261605 [details] proposed fix Clearing flags on attachment: 261605 Committed r190026: <http://trac.webkit.org/changeset/190026>
WebKit Commit Bot
Comment 3 2015-09-20 08:30:31 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.