Closed Bug 984269 Opened 6 years ago Closed 6 years ago

Rename nsDOMEventTargetHelper to mozilla::dom::EventTargetHelper or something

Categories

(Core :: DOM: Events, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla31

People

(Reporter: masayuki, Assigned: masayuki)

References

Details

Attachments

(1 file)

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)
Flags: needinfo?(bugs)
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.
Flags: needinfo?(bugs)
Attached patch PatchSplinter Review
Depending on nsEventStateManager -> mozilla::EventStateManager's patch.
Attachment #8398221 - Flags: review?(bugs)
Attachment #8398221 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/8560fc7e2849
Status: ASSIGNED → RESOLVED
Closed: 6 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.