Closed
Bug 417339
Opened 16 years ago
Closed 16 years ago
dynamic changes to xlink:href of svg:feImage don't cause update
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla1.9beta4
People
(Reporter: dbaron, Assigned: takenspc)
Details
(Keywords: testcase)
Attachments
(2 files)
3.86 KB,
application/xhtml+xml
|
Details | |
1.12 KB,
patch
|
longsonr
:
review+
roc
:
superreview+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
When I dynamically change the xlink:href of an svg:feImage, nothing gets updated. If I remove and reinsert the filter element, then things get updated. Steps to reproduce: 1. load attached testcase (simplified from layout/tools/reftest/reftest-analyzer.xhtml, which is working around this bug). 2. press "Toggle feImage". 3. press "Reinsert content" (repeat steps 2 and 3 as desired) Expected results: 2. the colored square on the right changes from green to red (or vice-versa) 3. nothing changes Actual results: 2. nothing changes 3. the colored square changes as it should have in step (2) (I haven't checked whether the spec says anything about dynamic changes; I'm just assuming that in general we're expected to update in response to them. If the SVG spec makes an exception here, I'm not aware of it.)
Reporter | ||
Comment 1•16 years ago
|
||
Assignee | ||
Comment 2•16 years ago
|
||
It seems a typo, not a feature.
Updated•16 years ago
|
Attachment #303667 -
Flags: review?(longsonr) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #303667 -
Flags: superreview?(tor)
![]() |
||
Updated•16 years ago
|
Attachment #303667 -
Flags: superreview?(tor) → superreview?(roc)
Attachment #303667 -
Flags: superreview?(roc) → superreview+
![]() |
||
Updated•16 years ago
|
Attachment #303667 -
Flags: approval1.9?
Updated•16 years ago
|
Attachment #303667 -
Flags: approval1.9? → approval1.9+
Updated•16 years ago
|
Keywords: checkin-needed
Comment 3•16 years ago
|
||
Checking in content/svg/content/src/nsSVGFilters.cpp; /cvsroot/mozilla/content/svg/content/src/nsSVGFilters.cpp,v <-- nsSVGFilters.cpp new revision: 1.66; previous revision: 1.65 done
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
OS: Linux → All
Hardware: PC → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta4
![]() |
||
Comment 4•16 years ago
|
||
Thanks for the fix taken!
You need to log in
before you can comment on or make changes to this bug.
Description
•