RWHI should implement BrowserAccessibilityDelegate

This is just refactoring, there should be no behavior changes.
I realized that most of the functionality of
BrowserAccessibilityDelegate was implemented by
RenderWidgetHostImpl, but each RenderWidgetHostView*
was actually implementing the interface, resulting in
extra code. It's simpler to have RWHI implement it and
just call into RWHV for the few cases where the view is
involved.

This also removes the parallel mac-only delegate, since
it was duplicating most of the same functionality.
Calling OriginInScreen is a couple of extra lines of code
because it needs to convert from NSPoint & NSSize to
gfx::Rect, but in every other case it's the same or simpler
this way.

BUG=none
NOTRY=true

Review URL: https://codereview.chromium.org/246773008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267097 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 0 insertions, 5 deletions

diff --git a/content/browser/accessibility/browser_accessibility_win.cc b/content/browser/accessibility/browser_accessibility_win.cc
index ac2af33..4e398c2 100644
--- a/content/browser/accessibility/browser_accessibility_win.cc
+++ b/content/browser/accessibility/browser_accessibility_win.cc
@@ -2819,11 +2819,6 @@ STDMETHODIMP BrowserAccessibilityWin::GetPatternProvider(PATTERNID id,
            << id;
   if (id == UIA_ValuePatternId || id == UIA_TextPatternId) {
     if (IsEditableText()) {
-      // The BrowserAccessibilityManager keeps track of instances when
-      // we don't want to show the on-screen keyboard.
-      if (!manager()->IsOSKAllowed(GetGlobalBoundsRect()))
-        return E_NOTIMPL;
-
       DVLOG(1) << "Returning UIA text provider";
       base::win::UIATextProvider::CreateTextProvider(true, provider);
       return S_OK;