666254 - nsIClipboard.emptyClipboard doesn't work

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect)

Description

[vb4guy]

User-Agent:       Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1; GTB7.0; .NET CLR 2.0.50727; .NET CLR 3.0.04506.30; .NET CLR 3.0.4506.2152; .NET CLR 3.5.30729; .NET CLR 1.1.4322; MS-RTC LM 8)
Build Identifier: Mozilla/5.0 (Windows NT 5.1; rv:5.0) Gecko/20100101 Firefox/5.0

According to https://developer.mozilla.org/en/nsIClipboard#emptyClipboard() calling the emptyClipboard method from nsIClipboard should clear the clipboard but it doesn't do so.  This is happening in Windows XP Pro.

Reproducible: Always

Steps to Reproduce:
Code: 
        this.ClearData = function()
        {
            netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
            
            var nsIClipboard = Components.classes["@mozilla.org/widget/clipboard;1"].getService(Components.interfaces.nsIClipboard); 
            nsIClipboard.emptyClipboard(nsIClipboard.kGlobalClipboard); // This doesn't appear to do anything
        };


Actual Results:  
The clipboard remains unmodified.

Expected Results:  
The clipboard should be empty.

Comment 1

[Marco Bonardo [:mak]]

it clears the logic clipboard, releasing the owner, but as specified in the idl it doesn't clear the native clipboard.

Comment 2

[vb4guy]

Marco,

Thanks for the quick response!

Looking at the documentation again I see the paragraph that states that, but is there some other way to clear the native clipboard?

-Mike

Comment 3

[Marco Bonardo [:mak]]

setData with a transferable that contains an unknown flavor should work, I don't know of other ways

Comment 4

[vb4guy]

I will try that but I think that this is a bug because the method takes the argument aWhichClipboard which can indicate the global/native clipboard.

Comment 5

[(no longer active)]

Hmm, the nsClipboard implementation in widget/cocoa doesn't even override EmptyClipboard(), so no wonder that function does not work on OSX (since we're using the nsBaseClipboard implementation which does not talk to the OS.)

Comment 6

[Jan Jongboom [:janjongboom] (inactive)]

According to https://bugzilla.mozilla.org/show_bug.cgi?id=416459#c15 this is a feature, not a bug...

Comment 7

[(no longer active)]

That's not true.  The Windows implementation does indeed clear the clipboard contents.

(Note that it's possible that there is no platform API on mac to do the same, I'm not very familiar with these APIs on mac.)

Comment 8

[Mirko Brodesser (:mbrodesser-Igalia), offline until April 25h]

Should this work nowadays? In bug 1261299 EmptyClipboard for Mac OS was implemented.

Comment 9

[Mirko Brodesser (:mbrodesser-Igalia), offline until April 25h]

@spohl: any idea? If it actually works, it could allow enabling existing tests, see https://searchfox.org/mozilla-central/search?q=%28666254%29&path=&case=true&regexp=true.spohl

Comment 10

[Makoto Kato [:m_kato]]

Attachment: Bug 666254 - nsIClipboard.emptyClipboard should clear system clipboard. r=#mac-reviewers

Actually, nsIClipboard.emptyClipboard on macOS doesn't clear system
clipboard. Since other platforms clear it, so I would like to change to same
behaviour.

Also, since we have base class of nsIClipboard, we should Inherit it to share
code.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c40c000f87d7
nsIClipboard.emptyClipboard should clear system clipboard. r=mac-reviewers,spohl

Comment 12

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/c40c000f87d7

Comment 13

[Chris Peterson [:cpeterson]]

Setting status-firefox100=wontfix. I don't think we need to uplift this fix to Beta 100 because this is a very old bug.