Closed
Bug 1673837
Opened 4 years ago
Closed 4 years ago
Convert IMPL_MUTATION_NOTIFICATION to an inline function
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
84 Branch
Tracking | Status | |
---|---|---|
firefox84 | --- | fixed |
People
(Reporter: saschanaz, Assigned: saschanaz)
Details
Attachments
(4 files)
The macro is too long and complex and thus hard to debug.
Assignee | ||
Updated•4 years ago
|
Assignee: nobody → krosylight
Updated•4 years ago
|
Priority: -- → P3
Updated•4 years ago
|
Severity: -- → N/A
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
Depends on D95352
Assignee | ||
Comment 3•4 years ago
|
||
Depends on D95353
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D95354
Updated•4 years ago
|
Attachment #9184847 -
Attachment description: Bug 1673837 - Part 4: Replace with IMPL_[NON]REMOVAL_NOTIFICATION r=smaug → Bug 1673837 - Part 4: Replace with Notify[Non]RemovalImpl r=smaug
Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0cd808252202 Part 1: Create nsDOMMutationEnterLeave RAII class r=smaug https://hg.mozilla.org/integration/autoland/rev/28f5fa941a18 Part 2: Extract PresShell notification as a separate macro r=smaug https://hg.mozilla.org/integration/autoland/rev/1f8af169aa3e Part 3: Create ForEachAncestorObserver() r=smaug https://hg.mozilla.org/integration/autoland/rev/f36f3ed54672 Part 4: Replace with Notify[Non]RemovalImpl r=smaug
Comment 6•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0cd808252202
https://hg.mozilla.org/mozilla-central/rev/28f5fa941a18
https://hg.mozilla.org/mozilla-central/rev/1f8af169aa3e
https://hg.mozilla.org/mozilla-central/rev/f36f3ed54672
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox84:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•