RWHI should implement BrowserAccessibilityDelegate

content/port/browser/render_widget_host_view_port.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_PORT_BROWSER_RENDER_WIDGET_HOST_VIEW_PORT_H_
 #define CONTENT_PORT_BROWSER_RENDER_WIDGET_HOST_VIEW_PORT_H_
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 309 matching lines @@
   virtual void SetPopupType(blink::WebPopupType popup_type) = 0;
   virtual blink::WebPopupType GetPopupType() = 0;
 
   // Get the BrowserAccessibilityManager if it exists, may return NULL.
   virtual BrowserAccessibilityManager*
       GetBrowserAccessibilityManager() const = 0;
   // Create a BrowserAccessibilityManager for this view if it's possible to
   // create one and if one doesn't exist already. Some ports may not create
   // one depending on the current state.
   virtual void CreateBrowserAccessibilityManagerIfNeeded() = 0;
-
+  virtual void SetBrowserAccessibilityManager(
+      BrowserAccessibilityManager* manager) = 0;

[jam, 2014/04/25 21:09:46]

this is an internal content class, yet it's being passed as an argument to a
port interface which can't happen. i.e. if someone was to implement
RenderWidgetHostViewPort in another repo, they wouldn't be able to access it.

+  virtual void OnAccessibilitySetFocus(int acc_obj_id) = 0;
+  virtual void AccessibilityShowMenu(int acc_obj_id) = 0;
+  virtual gfx::Point AccessibilityOriginInScreen(const gfx::Rect& bounds) = 0;
   // Return a value that is incremented each time the renderer swaps a new frame
   // to the view.
   virtual uint32 RendererFrameNumber() = 0;
   // Called each time the RenderWidgetHost receives a new frame for display from
   // the renderer.
   virtual void DidReceiveRendererFrame() = 0;
 
   virtual SkBitmap::Config PreferredReadbackFormat() = 0;
 
 #if defined(OS_MACOSX)
@@ skipping 16 matching lines @@
 
   // Returns an HWND that's given as the parent window for windowless Flash to
   // workaround crbug.com/301548.
   virtual gfx::NativeViewId GetParentForWindowlessPlugin() const = 0;
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PORT_BROWSER_RENDER_WIDGET_HOST_VIEW_PORT_H_