340822 - New ATK: Move accessible/public/atk interfaces to accessible/public

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

The following interfaces now need to be build cross-platform, in order for our new-atk code cleanup to move forward. These interfaces will now be available for JavaScript a11y implementations.

nsIAccessibleEditableText.idl
nsIAccessibleHyperLink.idl
nsIAccessibleHyperText.idl
nsIAccessibleTable.idl
nsIAccessibleText.idl
nsIAccessibleValue.idl

Another reason to do this is for the long term goal of bringing accessibility on Windows platforms much closer to what's on Linux.

Comment 1

[Aaron Leventhal]

Question: how do we move the files with CVS without destroying the CVS blame history?
The only actual change which requires a patch is to the makefiles. The rest is just a file move.

Comment 2

[Aaron Leventhal]

Attachment: These changes, plus move the idl files from accessible/public/atk up 1 directory

Comment 3

[Ginn Chen]

Comment on attachment 224854 [details] [diff] [review]
These changes, plus move the idl files from accessible/public/atk up 1 directory

changes of nsRootAccessible.cpp do not belong to this bug.

Comment 4

[Aaron Leventhal]

Attachment: Removed nsRootAccessible changes. This patch also requires the moving of the atk idl up one directory

Comment 5

[[e-mail]]

Comment on attachment 224982 [details] [diff] [review]
Removed nsRootAccessible changes. This patch also requires the moving of the atk idl up one directory

My understanding is that build changes don't require superreview but maybe you should run them past bryner or bsmedberg or someone.

Don't forget to file a dependent server operations bug on the CVS copies required.

Comment 6

[Aaron Leventhal]

Actually the CVS history only contains the initial checkin and a license change by gerv, so a cvs copy is not really necessary. We can do it via cvs add and cvs remove. Benjamin, do you agree?

Comment 7

[Benjamin Smedberg]

Comment on attachment 224982 [details] [diff] [review]
Removed nsRootAccessible changes. This patch also requires the moving of the atk idl up one directory

That's fine