Bug 323056 - [Accessibility] get_hyperlink returns incorrect address for accessible
Summary: [Accessibility] get_hyperlink returns incorrect address for accessible
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.6   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.6.1   Edit
Assignee: Carolyn MacLeod CLA
QA Contact:
URL:
Whiteboard:
Keywords: accessibility
Depends on:
Blocks:
 
Reported: 2010-08-18 11:49 EDT by Carolyn MacLeod CLA
Modified: 2011-01-10 12:42 EST (History)
1 user (show)

See Also:
Silenio_Quarti: review+

Attachments
Test with Snippet217a (5.50 KB, text/x-java)
2010-08-18 11:49 EDT, Carolyn MacLeod CLA 		no flags Details
Patch (913 bytes, patch)
2010-08-18 11:55 EDT, Carolyn MacLeod CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Carolyn MacLeod CLA 2010-08-18 11:49:36 EDT 
Created attachment 176912 [details]
Test with Snippet217a

Eclipse 3.6 on Windows.
Run the attached test snippet which shows a StyledText with 2 embedded controls.
Run AccProbe, and hover over the text in the StyledText.
Expand IAccessibleHypertext, and double-click on "hyperlink" to bring up the "getHyperlink" dialog.
Enter 0 or 1 for the hyperlink index, and click "Invoke Method".
NULL is returned. The expected return value is an accessible.
Comment 1 Carolyn MacLeod CLA 2010-08-18 11:55:42 EDT 
Created attachment 176913 [details]
Patch

The following one-line patch correctly returns the address for the accessible. Previously, the address was being returned in a VARIANT, but that is not how this method works. The address should be returned directly, in the specified address pointer.
Comment 2 Carolyn MacLeod CLA 2010-08-18 11:57:11 EDT 
SSQ, please review for 3.6.1. I will also release to 3.7 stream.
Comment 3 Carolyn MacLeod CLA 2010-08-18 12:02:52 EDT 
Fixed in 3.6.1 > 20100818.
Comment 4 Carolyn MacLeod CLA 2010-08-18 12:14:20 EDT 
Fixed in 3.7 > 20100818
Comment 5 Carolyn MacLeod CLA 2011-01-10 12:42:59 EST 
Changing from "Closed->Fixed" to "Resolved->Fixed".