Closed Bug 813019 Opened 9 years ago Closed 9 years ago

get rid of AccMutationEvent::mNode

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla20

People

(Reporter: surkov, Assigned: catalinn.iordache)

References

(Blocks 1 open bug)

Details

(Whiteboard: [good first bug][mentor=surkov.alexander@gmail.com][lang=c++])

Attachments

(1 file, 1 obsolete file)

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.
Assignee: nobody → catalinn.iordache
Attached file replaces (obsolete) —
Replace document->GetContainerAccessible(aEvent->mNode) with : aEvent->mAccessible->Parent()
Attachment #692596 - Flags: review?
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.
Attachment #692596 - Flags: review? → review?(trev.saunders)
Comment on attachment 692596 [details]
replaces

> 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.
Attachment #692596 - Flags: review?(trev.saunders)
i can't find AccMutationEvent class. Any hints?
(In reply to Catalin Iordache from comment #7)
> i can't find AccMutationEvent class. Any hints?

accessible/src/base/AccEvent.h
Attached patch bug-813019Splinter Review
mNode from AccEvent.h removed
Attachment #692596 - Attachment is obsolete: true
Attachment #693718 - Flags: review?(trev.saunders)
Attachment #693718 - Flags: review?(trev.saunders) → review+
Attachment #693718 - Flags: checkin?
Attachment #693718 - Flags: checkin?
you should use [checkin-needed] white board field not checkin? flag.
Whiteboard: [good first bug][mentor=surkov.alexander@gmail.com][lang=c++] → [checkin-needed][good first bug][mentor=surkov.alexander@gmail.com][lang=c++]
Actually, the checkin-needed keyword is what you really want :)
Keywords: checkin-needed
Whiteboard: [checkin-needed][good first bug][mentor=surkov.alexander@gmail.com][lang=c++] → [good first bug][mentor=surkov.alexander@gmail.com][lang=c++]
sorry for my mistakes, next time I'll know what to do.
https://hg.mozilla.org/mozilla-central/rev/3dcf9f0a42d8
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in before you can comment on or make changes to this bug.