Combine mai code into accessible/src/atk

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: yuanyi21, Assigned: yinbolian)

Tracking

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

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

16 years ago
this is a followup to bug 199060.
(Reporter)

Comment 1

16 years ago
This is the porting guide Aaron mentioned to me on AIM: 

1) put some platform code in the "Wrap" classes, 
2) don't have loose accessibles floating around that can't get Shutdown() when 
a dom ancestor above them is Shutdown(),  
3) take advantage of the integration of the Wrap classes with the accessible 
class hierarchy
4) make sure that any member variables with destructors outside the module get 
called at Shutdown(). Override Shutdown() for specific accessible classes to do 
this.
(Assignee)

Comment 2

16 years ago
Thanks, Kyle and Aaron.
Now, I beginning to work on moving the code. I found there are more changes
needed than I  thought. I will ask for help if there are problems.

-Bolian
Status: NEW → ASSIGNED
(Assignee)

Comment 3

16 years ago
Posted patch patchSplinter Review
This patch works but is not complete. New bugs will be filed to add new code
(Reporter)

Comment 4

16 years ago
Comment on attachment 121371 [details] [diff] [review]
patch

looks good! r=kyle
Attachment #121371 - Flags: review+
(Assignee)

Updated

16 years ago
Depends on: 203661
(Assignee)

Comment 5

16 years ago
Part of the mai code has been checked in trunk.  The interfaces part will be
moved in another bug (bug 203661)
(Assignee)

Updated

16 years ago
Depends on: 204574
(Reporter)

Comment 6

16 years ago
this was fixed long time ago.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.