Closed Bug 984269 Opened 9 years ago Closed 9 years ago
DOMEvent Target Helper to mozilla::dom::Event Target Helper or something
nsDOMEventTargetHelper doesn't represents standard DOM class. However, it's a base class of concrete classes derived from mozilla::dom::EventTarget. Therefore, for simpler code, it should be in mozilla::dom. However, "Helper" sounds odd to me. It sounds like *Utils or something external class of EventTarget. Is StubEventTarget or something better? (like nsStubMutationObserver)
I actually like the DOM prefix here, since we have also nsIEventTarget stuff which is all about threads and not about DOM Events. And the class does have some utils stuff, so *Helper isn't so weird to me. StubEventTarget might be ok too, though that indeed reminds more about StubMutationObserver which is a really dummy class (no-op methods). The current nsDOMEventTargetHelper does quite a bit more than that. (ns)DOMEventTargetHelper has also the nice acronym, DETH. So I'd probably call it mozilla::DOMEventTargetHelper.
Depending on nsEventStateManager -> mozilla::EventStateManager's patch.
Attachment #8398221 - Flags: review?(bugs)
Attachment #8398221 - Flags: review?(bugs) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Fixed merge bustage between this and another change: https://hg.mozilla.org/mozilla-central/rev/4941a2ac0786
(In reply to David Baron [:dbaron] (needinfo? me) (UTC-7) from comment #5) > Fixed merge bustage between this and another change: > https://hg.mozilla.org/mozilla-central/rev/4941a2ac0786 Thank you very much!
The following changesets are now in Firefox Nightly: > 8560fc7e2849 Bug 984269 Rename nsDOMEventTargetHelper to mozilla::DOMEventTargetHelper r=smaug > 4941a2ac0786 Fix merge bustage between changeset c8be375e0c41 (bug 988111) and changeset 8560fc7e2849 (bug 984269), merged (I think) in changeset 0ff6afce0133, on a CLOSED TREE. Nightly Build Information: ID: 20140402030201 Changeset: 4941a2ac0786109b08856738019b016a6c5a66a6 Version: 31.0a1 TBPL: https://tbpl.mozilla.org/?rev=4941a2ac0786 URL: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central Download Links: > Linux x86: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-i686.tar.bz2 > Linux x86_64: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-x86_64.tar.bz2 > Linux x86_64 ASAN: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-x86_64-asan.tar.bz2 > Mac: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.mac.dmg > Win32: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.win32.installer.exe > Win64: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.win64-x86_64.installer.exe Previous Nightly Build Information: ID: 20140401030203 Changeset: 1417d180a1d8665b1a91b897d1cc4cc31e7980d4 Version: 31.0a1 TBPL: https://tbpl.mozilla.org/?rev=1417d180a1d8 URL: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-01-03-02-03-mozilla-central
You need to log in before you can comment on or make changes to this bug.