<http://webkit.org/b/59755> AXObjectCache::attachWrapper() over-retains an Accessibil...
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Apr 2011 23:54:35 +0000 (23:54 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Apr 2011 23:54:35 +0000 (23:54 +0000)
commit2615b71c834c15e10e3d93334b89f499b6fa5de7
treef292031eedce3303c3a4d3cc0ccd717011915ed3
parent978898b3a623e965e227fc65826d636c9fe58d7d
<webkit.org/b/59755> AXObjectCache::attachWrapper() over-retains an AccessibilityObjectWrapper

Reviewed by Chris Fleizach.

Previously, AXObjectCache::attachWrapper over-retained each
AccessibilityObjectWrapper it created, which is why
AXObjectCache::detachWrapper() contained an extra -release.

* accessibility/mac/AXObjectCacheMac.mm:
(WebCore::AXObjectCache::detachWrapper): Removed extra -release.
(WebCore::AXObjectCache::attachWrapper): Because setWrapper()
assigns to a RetainPtr<> instance variable, it doesn't need a +1
retained object passed in.  Fix the leak by using a RetainPtr<>
for the AccessibilityObjectWrapper.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@85263 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/accessibility/mac/AXObjectCacheMac.mm