spun off bug 678454 comment #12:
Replace document->GetContainerAccessible(aEvent->mNode); on aEvent->mAccessible->Parent(). We can do that because CreateTextChangeEventFor is called before we mutate the tree.
Hi. I'm new here and I would like to work at this bug.
In which file I have to look for the GetContainerAccessible ?
(In reply to Catalin Iordache from comment #2)
> In which file I have to look for the GetContainerAccessible ?
you could use mxr tool to search things (for example, http://mxr.mozilla.org/mozilla-central/search?string=document-%3EGetContainerAccessible%28aEvent-%3EmNode%29;)
thank you for taking it.
Created attachment 692596 [details]
Replace document->GetContainerAccessible(aEvent->mNode) with : aEvent->mAccessible->Parent()
Catalin, you need to ask someone for review when you upload the patch (here you will find a list of peers for each component - https://wiki.mozilla.org/Modules/All). In this case I'd recommend Trevor for review.
Comment on attachment 692596 [details]
> NotificationController::CreateTextChangeEventFor(AccMutationEvent* aEvent)
> DocAccessible* document = aEvent->GetDocAccessible();
>- Accessible* container = document->GetContainerAccessible(aEvent->mNode);
>+ Accessible* container = aEvent->mAccessible->Parent();
you should remove the member from AccMutationEvent class as well.
i can't find AccMutationEvent class. Any hints?
(In reply to Catalin Iordache from comment #7)
> i can't find AccMutationEvent class. Any hints?
Created attachment 693718 [details] [diff] [review]
mNode from AccEvent.h removed
you should use [checkin-needed] white board field not checkin? flag.
Actually, the checkin-needed keyword is what you really want :)
sorry for my mistakes, next time I'll know what to do.