http://bugs.winehq.org/show_bug.cgi?id=13891
--- Comment #51 from Vincent Povirk madewokherd@gmail.com 2010-05-13 13:49:30 --- I don't think your patch is correct. winebrowser should never have to do a string conversion.
We have tests that show that in the client=ascii, server=unicode case the data passed to DdeClientTransaction is converted from ascii to unicode if it is not a unicode string. These tests are currently marked todo.
However, if they were fixed, winebrowser would do an additional conversion in this case, which would break it, even with your patch.