29124 2009-09-10 08:25:21 -0700 AccessibleBase::accSelect is not implemented at all 2011-03-31 16:27:50 -0700 1 1 1 Unclassified WebKit Accessibility 528+ (Nightly build) PC Windows XP RESOLVED DUPLICATE 33918 P2 Normal --- 0 mattc webkit-unassigned aroben jhoneycutt oldest_to_newest 146201 0 mattc 2009-09-10 08:25:21 -0700 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. 146202 1 mattc 2009-09-10 08:26:21 -0700 I'm working on a solution now. 146332 2 39378 mattc 2009-09-10 14:16:11 -0700 Created attachment 39378 Initial patch; only implements SELFLAG_TAKEFOCUS This patch has no test case; I'll submit that later. 146346 3 39378 jhoneycutt 2009-09-10 14:45:22 -0700 Comment on attachment 39378 Initial patch; only implements SELFLAG_TAKEFOCUS Looks good to me, but r- for the missing test as we discussed. Thanks for the patch, Matt! 377534 4 jhoneycutt 2011-03-31 16:27:50 -0700 Forward duping to 33918, since that bug is fixed. *** This bug has been marked as a duplicate of bug 33918 *** 39378 2009-09-10 14:16:11 -0700 2010-06-10 19:01:08 -0700 Initial patch; only implements SELFLAG_TAKEFOCUS ImplementTakeFocus.txt text/plain 1342 mattc SW5kZXg6IFdlYktpdC93aW4vQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC93aW4vQ2hh bmdlTG9nCShyZXZpc2lvbiA0ODI2NSkKKysrIFdlYktpdC93aW4vQ2hhbmdlTG9nCSh3b3JraW5n IGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMDktMDktMTAgIE1hdHQgQ2FtcGJlbGwgIDxtYXR0 Y0BzZXJvdGVrLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor ICAgICAgICBQYXJ0aWFsbHkgaW1wbGVtZW50IEFjY2Vzc2libGVCYXNlOjphY2NTZWxlY3QuCisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yOTEyNAorCisg ICAgICAgICogQWNjZXNzaWJsZUJhc2UuY3BwOgorICAgICAgICAoQWNjZXNzaWJsZUJhc2U6OmFj Y1NlbGVjdCk6IE9ubHkgdGhlIFNFTEZMQUdfVEFLRUZPQ1VTIGZsYWcgaXMKKyAgICAgICAgaW1w bGVtZW50ZWQgc28gZmFyLgorCiAyMDA5LTA5LTA3ICBEcmV3IFdpbHNvbiAgPGF0d2lsc29uQGdv b2dsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGF2aWQgTGV2aW4uCkluZGV4OiBXZWJL aXQvd2luL0FjY2Vzc2libGVCYXNlLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXQvd2luL0FjY2Vz c2libGVCYXNlLmNwcAkocmV2aXNpb24gNDgyNTEpCisrKyBXZWJLaXQvd2luL0FjY2Vzc2libGVC YXNlLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMzI2LDggKzMyNiwxOSBAQCBIUkVTVUxUIFNURE1F VEhPRENBTExUWVBFIEFjY2Vzc2libGVCYXNlCiAgICAgcmV0dXJuIFNfT0s7CiB9CiAKLUhSRVNV TFQgU1RETUVUSE9EQ0FMTFRZUEUgQWNjZXNzaWJsZUJhc2U6OmFjY1NlbGVjdChsb25nLCBWQVJJ QU5UKQorSFJFU1VMVCBTVERNRVRIT0RDQUxMVFlQRSBBY2Nlc3NpYmxlQmFzZTo6YWNjU2VsZWN0 KGxvbmcgZmxhZ3MsIFZBUklBTlQgdkNoaWxkKQogeworICAgIEFjY2Vzc2liaWxpdHlPYmplY3Qq IGNoaWxkT2JqOworICAgIEhSRVNVTFQgaHIgPSBnZXRBY2Nlc3NpYmlsaXR5T2JqZWN0Rm9yQ2hp bGQodkNoaWxkLCBjaGlsZE9iaik7CisKKyAgICBpZiAoRkFJTEVEKGhyKSkKKyAgICAgICAgcmV0 dXJuIGhyOworCisgICAgaWYgKGZsYWdzID09IFNFTEZMQUdfVEFLRUZPQ1VTKSB7CisgICAgICAg IGNoaWxkT2JqLT5zZXRGb2N1c2VkKHRydWUpOworICAgICAgICByZXR1cm4gU19PSzsKKyAgICB9 CisKICAgICByZXR1cm4gRV9OT1RJTVBMOwogfQogCg==