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.

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 on attachment 261788 [details] [diff] [review] patch r=me if compile warnings are fixed.

with ginn's comments, for checking in

checked in