144700 - To polish the nsIAccessibleHyperLink interface

Status: VERIFIED FIXED

(Core :: Disability Access APIs, defect)

Description

[Gilbert Fang]

In the mozilla/ accessible/ public/ nsIAccessibleHyperLink.idl ,
1, At line 34, readonly attribute long links;  I think this from the orignal  
"long getNAnchors ();", which means  the number of anchors in this link.
So, it is better that the attribute's name is  *anchors*.
2, For line 35 and 36, it is better to remove the first word "get" of the
attribute name, so that they sound like a attribute instead of a method.

Comment 1

[Gilbert Fang]

Hi , Bill and John 
Would you please spend some time to investigate  this? And because it is the
interface definition and most of my work will base on this, I am very eager to
know your opinion on it.

Comment 2

[Gilbert Fang]

Attachment: rename the attributes so that they sound right

need r=?

Comment 3

[Gilbert Fang]

Attachment: recreate the patch per kyle's suggestion

Kyle suggested that the initial letter of attribute should be lowercase.

Comment 4

[Kyle Yuan]

Comment on attachment 84009 [details] [diff] [review]
recreate the patch per kyle's suggestion

r=kyle.yuan

Comment 5

[Gilbert Fang]

see also bugsceri 75

Comment 6

[Johnny Stenback (:jst)]

Comment on attachment 84009 [details] [diff] [review]
recreate the patch per kyle's suggestion

sr=jst as long as aaronl or jgaunt ok this change.

Comment 7

[Aaron Leventhal]

r=aaronl

Comment 8

[Gilbert Fang]

thanks to aaronl and jst.

Comment 9

[Gilbert Fang]

Kyle has checked in the patch
http://bugzilla.mozilla.org/attachment.cgi?id=84009&action=view
Thanks. 

Comment 10

[Aaron Leventhal]

fixed

Comment 11

[dsirnapalli]

-- Checked nsIAccessibleHyperLink.idl in latest Mozilla trunk build. Required
changed are made. Marking the bug as verified.