Closed
Bug 479733
Opened 15 years ago
Closed 15 years ago
re-enable clipboard on WinCE
Categories
(Core :: Widget: Win32, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fennec | 1.0b1-wm+ | --- |
People
(Reporter: vlad, Assigned: vlad)
Details
Attachments
(1 file, 1 obsolete file)
11.13 KB,
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
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 | ||
Updated•15 years ago
|
Assignee: nobody → vladimir
Assignee | ||
Updated•15 years ago
|
Attachment #363634 -
Attachment is obsolete: true
Attachment #363634 -
Flags: review?(doug.turner)
Assignee | ||
Comment 1•15 years ago
|
||
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).
Updated•15 years ago
|
tracking-fennec: --- → 1.0b1-wm+
Comment 2•15 years ago
|
||
What is the reason for the patch here being marked obsolete?
Assignee | ||
Comment 3•15 years ago
|
||
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•15 years ago
|
||
Ugh, got faked out by tidybug which doesn't respect my comment sorting (so I saw only comment #0).
Assignee | ||
Comment 5•15 years ago
|
||
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•15 years ago
|
Attachment #372359 -
Flags: review? → review+
Comment 6•15 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?
Assignee | ||
Comment 7•15 years ago
|
||
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.
Assignee | ||
Comment 8•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/0fa3d40dbeb7
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•