The default bug view has changed. See this FAQ.

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

RESOLVED FIXED

Status

()

Core
Disability Access APIs
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Aaron Leventhal, Assigned: Aaron Leventhal)

Tracking

({access})

Trunk
x86
All
access
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

11 years ago
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.
(Assignee)

Comment 1

11 years ago
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.
(Assignee)

Comment 2

11 years ago
Created attachment 224854 [details] [diff] [review]
These changes, plus move the idl files from accessible/public/atk up 1 directory
Attachment #224854 - Flags: review?(ginn.chen)
(Assignee)

Updated

11 years ago
Blocks: 340827
(Assignee)

Updated

11 years ago
Blocks: 340825
(Assignee)

Updated

11 years ago
Blocks: 340829
(Assignee)

Updated

11 years ago
Blocks: 340833

Comment 3

11 years ago
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.
Attachment #224854 - Flags: review?(ginn.chen) → review-
(Assignee)

Comment 4

11 years ago
Created attachment 224982 [details] [diff] [review]
Removed nsRootAccessible changes. This patch also requires the moving of the atk idl up one directory
(Assignee)

Updated

11 years ago
Attachment #224982 - Flags: review?(ginn.chen)
(Assignee)

Updated

11 years ago
Attachment #224854 - Attachment is obsolete: true

Updated

11 years ago
Attachment #224982 - Flags: review?(ginn.chen) → review+
(Assignee)

Updated

11 years ago
Attachment #224982 - Flags: superreview?(neil)

Comment 5

11 years ago
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.
Attachment #224982 - Flags: superreview?(neil)
(Assignee)

Updated

11 years ago
Attachment #224982 - Flags: review?(benjamin)
(Assignee)

Comment 6

11 years ago
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 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
Attachment #224982 - Flags: review?(benjamin) → review+

Updated

11 years ago
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.