123933 - [Xlib] Clipboard is unuseable

Status: RESOLVED WONTFIX

(Core :: XUL, defect)

Description

[Roland Mainz]

Xlib widget's clipboard code is broken in various ways, we have to rewrite it...

Comment 1

[Louie Zhao]

Hi, Roland

   I have almost finished rewriting xlib clipboard code, and can attach the
patch after a few days. You can assign this bug to me.

--Louie
([e-mail])

Comment 2

[Roland Mainz]

Louie Zhao wrote:
> I have almost finished rewriting xlib clipboard code, and can attach the
> patch after a few days.

Wow!
Thanks! :))

> You can assign this bug to me.

Done!

Comment 3

[Louie Zhao]

Attachment: rewriting of nsClipboard.cpp and nsClipboard.h of xlib

Hi, Roland

 I have rewritten the xlib clipboard which implement almost all the features
gtk clipboard has. You can try & test it, thanks.

 Two things can be done to make this code better.

/* TODO:
 * 1. using INCR when writing data has some problem.  
 * no INCR when data size < SEL_DELIMIT, so set
 * SEL_DELIMIT to large number to avoid INCR writing	 
 * 
 * 2. when other app paste, don't support "TARGETS" 
 * request to get all "TARGET"s in mozilla's copy data.
 * "TARGETS" request of internal paste works well.
 */
 
 The two inperfection don't affect the using of xlib clipboard
 . Anyway, adding implementation of them will be better.

 BTW, who do you think suitable for reviewing and superviewing
 these code?

Comment 4

[Roland Mainz]

Louie Zhao wrote:
> Created an attachment (id=85577)
> rewriting of nsClipboard.cpp and nsClipboard.h of xlib
>
> I have rewritten the xlib clipboard which implement almost all the features
> gtk clipboard has.

WOW! Thanks! :)

> You can try & test it, thanks.
[snip]
> BTW, who do you think suitable for reviewing and superviewing
> these code?

Per http://bugzilla.mozilla.org/survey.cgi?action=list I am a module "peer"
(which means I can give you an r= or can say which r= is sufficient for me) and
since this code is not part of the default build we do not need sr= ... :)

I am reviewing this patch this weekend...

In the meantime:
- Did you test the patch with Rational Purify (my Purify still does not work...
;-( ) ?
- Does the patch work in the en_US.UTF-8 and the gb18030 locales (using the CDE
desktop) ?

Comment 5

[Louie Zhao]

I test the patch in the the en_US.UTF-8 and the gb18030 locales using the CDE
desktop. It works well. I have tested:
   1. copy webpage from another navigator(mozilla on gtk) to my composer(mozilla
on xlib)
   2. copy webpage from my navigator(mozilla on xlib) to another
composer(mozilla on gtk).
   3. copy webpage from my navigator(xlib) to my composer(xlib)
   These testings include copy & paste different kinds of data externally and
internally, I think it's comprehensive.

I have no Rational Purify on my machine, I can try later.

Comment 6

[Louie Zhao]

Hi, Roland
 Because I am busy recently, I am afraid I have no time to install and learn
Rational Purify(:-(I didn't use it before). If your Rational Purify can work,
can you test it? Thanks.

Comment 7

[Alexander Sack]

What happened to this bug (critical)? Has the clipboard code since then been fixed? 

Comment 8

[Alexander Sack]

Louie, does this patch still apply? Maybe you should request review then ... or make it apply again before.

Comment 9

[Mats Palmgren (inactive)]

Xlib port has been removed from trunk (bug 326152)

-> WONTFIX