Closed Bug 202085 Opened 23 years ago Closed 23 years ago

Combine mai code into accessible/src/atk

Categories

(Core :: Disability Access APIs, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: yuanyi21, Assigned: yinbolian)

References

Details

Attachments

(1 file)

this is a followup to bug 199060.
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.
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
Attached patch patch — — Splinter Review
This patch works but is not complete. New bugs will be filed to add new code
Comment on attachment 121371 [details] [diff] [review] patch looks good! r=kyle
Attachment #121371 - Flags: review+
Depends on: 203661
Part of the mai code has been checked in trunk. The interfaces part will be moved in another bug (bug 203661)
Depends on: 204574
this was fixed long time ago.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: