282285 - When creating DDEHandle, should use CF_UNICODETEXT when available

Status: RESOLVED WONTFIX

(Toolkit :: Startup and Profile System, defect)

Description

[Jungshik Shin]

a spin-off of bug 281781

When calling DDECreateDataHandle, we'd better use CF_UNICODETEXT on Win 2k/XP.

http://lxr.mozilla.org/seamonkey/source/toolkit/xre/nsNativeAppSupportWin.cpp#1297

http://msdn.microsoft.com/library/default.asp?url=/library/en-us/winui/winui/windowsuserinterface/dataexchange/dynamicdataexchangemanagementlibrary/dynamicdataexchangemanagementreference/dynamicdataexchangemanagementfunctions/ddecreatedatahandle.asp


http://msdn.microsoft.com/library/default.asp?url=/library/en-us/winui/winui/windowsuserinterface/dataexchange/clipboard/clipboardformats.asp

Comment 1

[Jungshik Shin]

*** Bug 334282 has been marked as a duplicate of this bug. ***

Comment 2

[Jungshik Shin]

In addition, DDECreateStringHandle should use CP_WINUNICODE instead of CP_WINANSI.

Comment 3

[Benjamin Smedberg]

We need to be very careful to retain backwards compatibility with the old DDE format. It is not sufficient to simply replace our DDE message with a widechar version.

Comment 4

[Edward Leigh]

I've been waiting a long time for Firefox's DDE to be Unicode-aware so that it would work properly with my bookmark manager (www.bookmarkbuddy.net). Backward compatibility is simple to maintain: the calling function can specify either CF_TEXT or CF_UNICODETEXT in the WM_DDE_REQUEST message. Firefox should expect the input string to be in the nominated format, and should return data (e.g. in response to WWW_GetWindowInfo) in the nominated format.

Incidentally it would be nice too if Firefox responded to the WWW_ListWindows topic, dropped after Netscape v.4!

Comment 5

[Robert Strong (they/them - no direct email)]

DDE is no longer used for shell integration so moving to component where this code lives.

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:mossop, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 7

[Masatoshi Kimura [:emk]]

We removed DDE support long ago (bug 513742).