Bug 602256 made both paths there do the same thing in some cases so it is probably possible to reduce the duplicate code there, or just replace the whole function with a simpler function.
Dave, you want to take this, or should I?
I probably wouldn't get around to it for a while
Assignee: nobody → bzbarsky
Priority: -- → P2
Created attachment 535599 [details] [diff] [review] Patch v1 Something like this, perhaps?
Assignee: bzbarsky → Ms2ger
Status: NEW → ASSIGNED
Attachment #535599 - Flags: review?(bzbarsky)
Comment on attachment 535599 [details] [diff] [review] Patch v1 Yep, like that.
Attachment #535599 - Flags: review?(bzbarsky) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
You need to log in before you can comment on or make changes to this bug.