[Mac, iOS] AccessibilityController doesn't uninstall global notification handler
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 20 Sep 2015 15:30:09 +0000 (15:30 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 20 Sep 2015 15:30:09 +0000 (15:30 +0000)
commit4077ed3945e3a1e3fa95f08c968b85250df516d3
tree89762d67aaab83a72a4ed63d5a2bc0456c155f92
parent69aa4075ed394aab7513dd3a3684b2d0718cdee0
[Mac, iOS] AccessibilityController doesn't uninstall global notification handler
https://bugs.webkit.org/show_bug.cgi?id=149384

Reviewed by Chris Fleizach.

Tools:

* DumpRenderTree/ios/AccessibilityControllerIOS.mm:
(AccessibilityController::addNotificationListener): Fixed a leak, and cleaned up the code.
(AccessibilityController::platformResetToConsistentState): Actually remove the handler,
regardless of whether someone else holds a reference (we also call -stopListening in
-dealloc).

* DumpRenderTree/mac/AccessibilityControllerMac.mm:
(AccessibilityController::platformResetToConsistentState):
(AccessibilityController::addNotificationListener):
Ditto.

LayoutTests:

* accessibility/mac/loaded-notification.html: Cleaned up the test - js-test-pre is
incompatible with directly using waitUntilDone.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@190026 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/accessibility/mac/loaded-notification.html
Tools/ChangeLog
Tools/DumpRenderTree/ios/AccessibilityControllerIOS.mm
Tools/DumpRenderTree/mac/AccessibilityControllerMac.mm