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
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...