There are a few spots throughout the code base that try to use nsDependent(C)String as if it were PromiseFlat(C)String, which it is not. They usually get lucky because the string being promised turned out to be flat, but we should just remove the bogus constructor (after fixing all the callers).
Created attachment 572291 [details] [diff] [review] Patch v1 https://tbpl.mozilla.org/?tree=Try&rev=6bf98ab2ac2b
Assignee: nobody → atulagrwl
Status: NEW → ASSIGNED
Attachment #572291 - Flags: review?(benjamin)
Comment on attachment 572291 [details] [diff] [review] Patch v1 Build busted in Windows.
(In reply to Atul Aggarwal from comment #2) > (From update of attachment 572291 [details] [diff] [review]) > Build busted in Windows. I didn't think the external API had the problem anyway.
Totally true. Verified now and I thought it was due to my change :P. Neil, shouldn't we close this issue if this is not relevant any more otherwise another fool like me will waste some time working on it :)
Looks like I filed this as a duplicate by mistake...
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 611503
You need to log in before you can comment on or make changes to this bug.