Closed Bug 1459401 Opened 2 years ago Closed 2 years ago

Remove unused ContentRemoved argument.

Categories

(Core :: DOM: Events, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: emilio, Assigned: emilio)

Details

Attachments

(1 file)

Looks like I forgot to do this when I was cleaning up notification arguments.
Assignee: nobody → emilio
Comment on attachment 8973440 [details]
Bug 1459401: Remove unused container argument in EventStateManager::ContentRemoved.

https://reviewboard.mozilla.org/r/241846/#review247744

::: commit-message-21ae2:3
(Diff revision 1)
> +Bug 1459401: Remove unused container argument in EventStateManager::ContentRemoved. r?smaug
> +
> +Can we assert PresShell::mDocument == ESM::mDocument? If so, we can remove the

Make sure to remove this sentence before pushing.

Those should be the same document, except during unlinking ESM's mDocument may be set to null.
And initially ESM's mDocument is null.
Attachment #8973440 - Flags: review?(bugs) → review+
Pushed by ecoal95@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b1628ac71fcc
Remove unused container argument in EventStateManager::ContentRemoved. r=smaug
https://hg.mozilla.org/mozilla-central/rev/b1628ac71fcc
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.