Migrate accessibility from RenderView to RenderFrame.

content/browser/renderer_host/legacy_render_widget_host_win.cc

 // Copyright (c) 2014 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 "content/browser/renderer_host/legacy_render_widget_host_win.h"
 
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/win/windows_version.h"
 #include "content/browser/accessibility/browser_accessibility_manager_win.h"
@@ skipping 52 matching lines @@
     ::EnableWindow(hwnd(), FALSE);
   } else {
     ::EnableWindow(hwnd(), TRUE);
   }
 }
 
 HWND LegacyRenderWidgetHostHWND::GetParent() {
   return ::GetParent(hwnd());
 }
 
-void LegacyRenderWidgetHostHWND::OnManagerDeleted() {
-  manager_ = NULL;
+void LegacyRenderWidgetHostHWND::OnManagerDeleted(
+    content::BrowserAccessibilityManagerWin* manager) {
+  if (manager_ == manager)

[David Tseng, 2014/07/14 19:08:02]

Ok; I see why you needed manager to be passed in here.

I'm just not entirely clear on the ownership relationship here. It seems like
maybe BAM should own LRWHW.

[dmazzoni, 2014/07/15 07:55:04]

Yeah, it used to - and then LRWHW ended up being used by RWHVA for other
purposes and that messed up the lifecycle. The change I pointed to earlier will
clean this up.

+    manager_ = NULL;
 }
 
 void LegacyRenderWidgetHostHWND::Show() {
   ::ShowWindow(hwnd(), SW_SHOW);
 }
 
 void LegacyRenderWidgetHostHWND::Hide() {
   ::ShowWindow(hwnd(), SW_HIDE);
 }
 
@@ skipping 280 matching lines @@
   // generate the legacy WM_VSCROLL/WM_HSCROLL messages.
   // We add these styles to ensure that trackpad/trackpoint scrolling
   // work.
   long current_style = ::GetWindowLong(hwnd(), GWL_STYLE);
   ::SetWindowLong(hwnd(), GWL_STYLE,
                   current_style | WS_VSCROLL | WS_HSCROLL);
   return 0;
 }
 
 }  // namespace content