Bug 29124 - AccessibleBase::accSelect is not implemented at all
Summary: AccessibleBase::accSelect is not implemented at all
Status: RESOLVED DUPLICATE of bug 33918
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-10 08:25 PDT by Matt Campbell
Modified: 2011-03-31 16:27 PDT (History)
2 users (show)

See Also:


Attachments
Initial patch; only implements SELFLAG_TAKEFOCUS (1.31 KB, patch)
2009-09-10 14:16 PDT, Matt Campbell
jhoneycutt: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Campbell 2009-09-10 08:25:21 PDT
Overview:
The accSelect method of the MSAA IAccessible interface is used by assistive technologies to manage both the focus and (where applicable) the selection.  The use of this method to manage the focus is especially important for screen readers which usually implement an alternative user interface for navigating a web page, often called a virtual buffer.  Specifically, the screen reader must be able to programmatically set the focus to elements in the web page as the user navigates.

Steps to reproduce (e.g. in a test program):
1.  Get an IAccessible object from WebKit using AccessibleObjectFromWindow.
2.  Call the accSelect method with SELFLAG_TAKEFOCUS as the first argument. (SELFLAG_TAKEFOCUS is a constant in oleacc.h)

Actual result:
The method returns the E_NOTIMPL COM error.

Expected result:
This method should at least handle the SELFLAG_TAKEFOCUS flag.

Reproduced with r48199 on Windows XP.
Comment 1 Matt Campbell 2009-09-10 08:26:21 PDT
I'm working on a solution now.
Comment 2 Matt Campbell 2009-09-10 14:16:11 PDT
Created attachment 39378 [details]
Initial patch; only implements SELFLAG_TAKEFOCUS

This patch has no test case; I'll submit that later.
Comment 3 Jon Honeycutt 2009-09-10 14:45:22 PDT
Comment on attachment 39378 [details]
Initial patch; only implements SELFLAG_TAKEFOCUS

Looks good to me, but r- for the missing test as we discussed. Thanks for the patch, Matt!
Comment 4 Jon Honeycutt 2011-03-31 16:27:50 PDT
Forward duping to 33918, since that bug is fixed.

*** This bug has been marked as a duplicate of bug 33918 ***