re-enable clipboard on WinCE

RESOLVED FIXED

Status

()

Core
Widget: Win32
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: vlad, Assigned: vlad)

Tracking

Trunk
All
Windows CE
Points:
---

Firefox Tracking Flags

(fennec1.0b1-wm+)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 363634 [details] [diff] [review]
add include and some stub code for CF_HDROP

The needed defines are in winuserm.h.

We should write the CF_HDROP code at some point, but it's not crucial for right now.  This is on Windows CE only, though I think that the same would work on Windows Mobile... winuserm.h is there as well.  I can enable it everywhere if that's ok.
Attachment #363634 - Flags: review?(doug.turner)
Assignee: nobody → vladimir
Attachment #363634 - Attachment is obsolete: true
Attachment #363634 - Flags: review?(doug.turner)
Comment on attachment 363634 [details] [diff] [review]
add include and some stub code for CF_HDROP

Never mind; I forgot to reenable the constructors and modules in WinWidgetFactory, so a lot of this ended up being dead code.  Our current clipboard code relies on the OLE clipboard, which CE/Mobile doesn't have.

I have another patch coming up that just stubs out the clipboard -- this is needed so that things like about:config don't crap out (since they unconditionally get the clipboard helper service and the like).
tracking-fennec: --- → 1.0b1-wm+

Comment 2

9 years ago
What is the reason for the patch here being marked obsolete?
Due to comment #1 -- the code was never actually being built, it was being stripped by the linker as dead code :)  I'll post the stub patch here shortly (looks like I just forgot); it should be straightforward to fill it out with support for the most straightforward TEXT copy/paste.

Comment 4

9 years ago
Ugh, got faked out by tidybug which doesn't respect my comment sorting (so I saw only comment #0).
Created attachment 372359 [details] [diff] [review]
stub out clipboard for CE, also reenable nsSound

This patch stubs out the clipboard for windows ce, and also reenables nsSound (the PURGE flag isn't supported even on the desktop, not clear what it should be doing).

Having this patch stops things like about:config from throwing when they try to access the clipboard service without checking whether they got it or not.
Attachment #372359 - Flags: review?

Updated

9 years ago
Attachment #372359 - Flags: review? → review+

Comment 6

9 years ago
Comment on attachment 372359 [details] [diff] [review]
stub out clipboard for CE, also reenable nsSound

part of this looks like bug 475361, which should land soon.  When you push this, please mark that bug fixed.

drop the changes in:
widget/src/windows/nsClipboard.cpp

in widget/src/windows/nsClipboardCE.h, why do we need:

+#include <windows.h>


I do not think this is enough stuff to actually get clipboard support working, right?  I have patches in Mozilla 1.8 that implement the missing clipboard functionality on WinCE.  Do you want to file a follow up?
Oh, I forgot that the idle stuff is in here -- bug 475361's fix seems more robust, so I'll take out the idle timer pieces from here.

I'll nuke windows.h, not needed.

And yeah -- this just stubs everything out.  We should do a followup to actually fill in the stubbed functions.
http://hg.mozilla.org/mozilla-central/rev/0fa3d40dbeb7
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.