Fix a few lingering bugs in BrowserAccessibilityManager and BrowserAccessibilityCocoa.

content/browser/accessibility/browser_accessibility_cocoa.mm

Index: content/browser/accessibility/browser_accessibility_cocoa.mm
diff --git a/content/browser/accessibility/browser_accessibility_cocoa.mm b/content/browser/accessibility/browser_accessibility_cocoa.mm
index ff9aed5c57d3fa51d405693cd3061c47b1c79b20..7b2c12c3933032f04f4d927500897c91d129c81f 100644
--- a/content/browser/accessibility/browser_accessibility_cocoa.mm
+++ b/content/browser/accessibility/browser_accessibility_cocoa.mm
@@ -246,9 +246,11 @@ NSDictionary* attributeToMethodNameMap = nil;
   return self;
 }
 
+extern "C" void NSAccessibilityUnregisterUniqueIdForUIElement(id element);

[dmazzoni, 2011/08/02 06:19:35]

Is this not in any include files?

Does it maybe belong in base/mac/cocoa_protocols or something like that? Maybe
ask Nico or Stuart or one of the other Mac devs. If all else fails, put it at
the top of the file and add a comment explaining why it's forward-declarered -
like if it was only defined in 10.6.

[David Tseng, 2011/08/02 21:30:57]

I couldn't find it in any header files. I'm guessing it's defined in a framework
dylib we link against?  I'll add Nico for comment.
On 2011/08/02 06:19:35, Dominic Mazzoni wrote:

 // Deletes our associated BrowserAccessibilityMac.
 - (void)dealloc {
   if (browserAccessibility_) {
+    NSAccessibilityUnregisterUniqueIdForUIElement(self);
     delete browserAccessibility_;
     browserAccessibility_ = NULL;
   }
@@ -832,5 +834,9 @@ NSDictionary* attributeToMethodNameMap = nil;
   return browserAccessibility_->renderer_id();
 }
 
+- (BOOL)accessibilityShouldUseUniqueId {
+  return YES;
+}
+
 @end