Closed
Bug 1405785
Opened 7 years ago
Closed 7 years ago
Some SVG rendering observers cleanup patches
Categories
(Core :: SVG, enhancement, P4)
Core
SVG
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| firefox57 | --- | wontfix |
People
(Reporter: jwatt, Assigned: jwatt)
Details
Attachments
(3 files)
|
6.65 KB,
patch
|
longsonr
:
review+
|
Details | Diff | Splinter Review |
|
17.23 KB,
patch
|
longsonr
:
review+
|
Details | Diff | Splinter Review |
|
3.88 KB,
patch
|
longsonr
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Updated•7 years ago
|
status-firefox57:
--- → wontfix
|
|
Assignee | |
Comment 1•7 years ago
|
||
Attachment #8915253 -
Flags: review?(longsonr)
|
|
Assignee | |
Comment 2•7 years ago
|
||
Attachment #8915254 -
Flags: review?(longsonr)
|
|
Assignee | |
Comment 3•7 years ago
|
||
Attachment #8915255 -
Flags: review?(longsonr)
|
||
Updated•7 years ago
|
Attachment #8915253 -
Flags: review?(longsonr) → review+
|
|
||
Updated•7 years ago
|
Attachment #8915254 -
Flags: review?(longsonr) → review+
|
|
||
Updated•7 years ago
|
Attachment #8915255 -
Flags: review?(longsonr) → review+
Pushed by jwatt@jwatt.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/f46ed2816658 part 1 - Rename nsSVGRenderingObserver::Start/StopListening. r=longsonr https://hg.mozilla.org/integration/mozilla-inbound/rev/0f1d6c8dc992 part 2 - Document and rename nsSVGRenderingObserver::DoUpdate to OnRenderingChange. r=longsonr https://hg.mozilla.org/integration/mozilla-inbound/rev/9602570c97b4 part 3 - Document and rename nsSVGRenderingObserver::InvalidateViaReferencedElement to OnNonDOMMutationRenderingChange. r=longsonr
|
|
Assignee | |
Updated•7 years ago
|
Keywords: leave-open
|
||
Comment 5•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/f46ed2816658 https://hg.mozilla.org/mozilla-central/rev/0f1d6c8dc992 https://hg.mozilla.org/mozilla-central/rev/9602570c97b4
|
|
Assignee | |
Comment 6•7 years ago
|
||
I have more patches that need cleaned up and landed, but I guess at this point I should open a new bug for that.
You need to log in
before you can comment on or make changes to this bug.
Description
•