The mistaken use of PRUint8 in some places means that the new SVG list item DOM wrappers will get confused if there are more than 256 items in the list.
Requesting approval at same time as review. Will add test too.
This isn't a read/write bad memory issue, but we should probably still get it in.
(In reply to comment #2) > This isn't a read/write bad memory issue Well, actually it is, but what I meant is that in this case it isn't an sg issue.
blocking2.0: --- → final+
Pushed to try: http://tbpl.mozilla.org/?tree=MozillaTry&rev=a360688c9ea0
The Try orange is due to bug 631939, so this one should be fine.
Pushed to try with fixed patch for bug 631939: http://tbpl.mozilla.org/?tree=MozillaTry&rev=bd7c865af7fa
(Comment 7 meant to say "with fixed patch for *bug 631946*) Comment 7 passed try-server (after some RelEng woes were resolved in bug 632211). --> Adding checkin-needed flag, at jwatt's request.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.