Reproduction: nsString mCiteReference; mCiteReference.AssignWithConversion("mid:A1/CC04netscapecom"); mCiteReference.ReplaceSubstring(NS_ConvertToString("/"), NS_ConvertToString("%2F")); Actual result: mCiteReference holds "mid:A1%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F" Expected result: mCiteReference holds "mid:A1%2FCC04netscapecom"

Bug 36656 is related.

rickg on sabbatical, setting assignee to syd since he has related bug. syd, please correct if you are the wrong person.

CC:ing scc@mozilla.org since this is a string issue, and there has been no activity in this bug since it was filed in june.

Reassigning to scc since syd hasn't looked at it and it's not on his table anyway. Changing component to string.

These are the 0.9 bugs that didn't make it, and that I intend to fix in 0.9.1

re-targeting milestones, starting from a clean slate

marking the functionality blockers for immediate attention in mozilla0.9.1

if it hasn't landed, let's land this in 0.9.2. let drivers know if this creates problems. thanks.

The fix to this bug is that we're going to replace the entire implementation of |ReplaceSubstring| with the new |nsSubstituteC?String| generator. See bug #82456. There's no perfect way to mark this bug for this resolution (it's not FIXED yet, it's not INVALID, it's not WONTFIX). We'll settle for making it a dup of the new bug. *** This bug has been marked as a duplicate of 82456 ***