Closed Bug 1422626 Opened 7 years ago Closed 7 years ago

class EnterLeaveDispatcher is declared twice

Categories

(Core :: DOM: Events, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: Sylvestre, Assigned: maiquynhtruong, Mentored)

References

(Blocks 1 open bug)

Details

(Whiteboard: [good first bug][lang=C++])

Attachments

(1 file)

A trivial good first bug to understand the workflows. found by clang-tidy "no definition found for 'EnterLeaveDispatcher', but a definition with the same name 'EnterLeaveDispatcher' found in another namespace 'mozilla'" The first declaration can be removed.
Blocks: 1422628
I'll try to take a stab on this. Do you mean removing the line: https://searchfox.org/mozilla-central/source/dom/events/EventStateManager.h#29 ?
Indeed! Please submit a patch for this!
Assignee: nobody → maitruong2801
Comment on attachment 8934409 [details] Bug 1422626 - Remove redundant declarion of class EnterLeaveDispatcher. Good, you should now find the official reviewer!
Attachment #8934409 - Flags: review?(sledru) → feedback+
(In reply to Sylvestre Ledru [:sylvestre] from comment #4) > Comment on attachment 8934409 [details] > Bug 1422626 - Remove redundant declarion of class EnterLeaveDispatcher. > > Good, you should now find the official reviewer! I'll probably need help with this. How do I find the official reviewers? I looked at the change log (https://hg.mozilla.org/mozilla-central/log/tip/dom/events/EventStateManager.cpp) and there are multiple people making changes to the file. Can I choose anyone of them? Thanks!
First, you should look at https://wiki.mozilla.org/Modules/All to find out who are the official reviewers of this component. Just look for "dom/events" on this page!
Attachment #8934409 - Flags: review?(sledru)
Attachment #8934409 - Flags: review?(masayuki)
Attachment #8934409 - Flags: review?(bugs)
Comment on attachment 8934409 [details] Bug 1422626 - Remove redundant declarion of class EnterLeaveDispatcher. One reviewer is enough!
Attachment #8934409 - Flags: review?(sledru)
Attachment #8934409 - Flags: review?(bugs)
Comment on attachment 8934409 [details] Bug 1422626 - Remove redundant declarion of class EnterLeaveDispatcher. https://reviewboard.mozilla.org/r/205328/#review212180
Attachment #8934409 - Flags: review?(masayuki) → review+
Pushed by sledru@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c03e9d114143 Remove redundant declarion of class EnterLeaveDispatcher. r=masayuki
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: