377756 - move property change events code to atk

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

Attachment: patch

Now atk code has support for PROPERTY_CHANGED event, but it is not used in MSAA and IA2 and we do not generate it in gecko therefore I moved this code into atk entirely.

Comment 1

[Ginn Chen]

2 compile warnings:
1) 
nsDocAccessibleWrap.cpp:90: warning: unused variable ‘oldAccWrap’
nsDocAccessibleWrap.cpp:90: warning: unused variable ‘newAccWrap

2)
nsAccessibleEventData.h: In constructor ‘nsAccEvent::nsAccEvent(PRUint32, nsIAccessible*, void*)’:
nsAccessibleEventData.h:66: warning: ‘nsAccEvent::mAccessible’ will be initialized after
nsAccessibleEventData.h:62: warning:   ‘void* nsAccEvent::mEventData’
nsAccessibleEventData.cpp:47: warning:   when initialized here
nsAccessibleEventData.h: In constructor ‘nsAccEvent::nsAccEvent(PRUint32, nsIDOMNode*, void*)’:
nsAccessibleEventData.h:67: warning: ‘nsAccEvent::mDOMNode’ will be initialized after
nsAccessibleEventData.h:62: warning:   ‘void* nsAccEvent::mEventData’
nsAccessibleEventData.cpp:53: warning:   when initialized here

and I happen to find a typo in patch of Bug 377617, I'm sorry I missed it.

+nsAccTextChangeEvent::
+  nsAccTextChangeEvent(nsIAccessible *aAccessible,
+                       PRInt32 aStart, PRUint32 aLength, PRBool aIsInserted):
+  nsAccEvent(::nsIAccessibleEvent::EVENT_TEXT_CHANGED, aAccessible, nsnull),
+  mStart(aStart), mLength(aLength), mIsInserted(mIsInserted)

should be mIsInserted(aIsInserted)

I'll double check this patch tomorrow.

Comment 2

[Ginn Chen]

Comment on attachment 261788 [details] [diff] [review]
patch

r=me if compile warnings are fixed.

Comment 3

[alexander :surkov (:asurkov)]

Attachment: patch2

with ginn's comments, for checking in

Comment 4

[alexander :surkov (:asurkov)]

checked in