Fix a few lingering bugs in BrowserAccessibilityManager and BrowserAccessibilityCocoa.

chrome/browser/renderer_host/render_widget_host_view_mac.mm

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <QuartzCore/QuartzCore.h>
 
 #include "chrome/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
@@ skipping 1045 matching lines @@
   if (render_widget_host_)
     render_widget_host_->Send(new ViewMsg_SetBackground(
         render_widget_host_->routing_id(), background));
 }
 
 void RenderWidgetHostViewMac::OnAccessibilityNotifications(
     const std::vector<ViewHostMsg_AccessibilityNotification_Params>& params) {
   if (!browser_accessibility_manager_.get()) {
     // Use empty document to process notifications
     webkit_glue::WebAccessibility empty_document;
+    // Renderer id's always start at 1000 as determined by webkit. Boot strap
+    // our ability to reuse BrowserAccessibility instances.
+    empty_document.id = 1000;

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

Could you move this to a common place, so it isn't duplicated in Mac and Windows
code? How about a static method in BrowserAccessibilityManager called
GetEmptyDocument or PopulateEmptyDocument?

     empty_document.role = WebAccessibility::ROLE_WEB_AREA;
     empty_document.state = 0;
     browser_accessibility_manager_.reset(
         BrowserAccessibilityManager::Create(cocoa_view_, empty_document, NULL));
   }
   browser_accessibility_manager_->OnAccessibilityNotifications(params);
 }
 
 void RenderWidgetHostViewMac::SetTextInputActive(bool active) {
   if (active) {
@@ skipping 1552 matching lines @@
   if (!string) return NO;
 
   // If the user is currently using an IME, confirm the IME input,
   // and then insert the text from the service, the same as TextEdit and Safari.
   [self confirmComposition];
   [self insertText:string];
   return YES;
 }
 
 @end