Implement nsWindow::SetIcon for windows ce




Widget: Win32
9 years ago
7 years ago


(Reporter: blassey, Unassigned)



Firefox Tracking Flags

(Not tracked)



(1 attachment)


9 years ago
tracking-fennec: --- → 1.0-wm+

Comment 1

9 years ago
Created attachment 367815 [details] [diff] [review]
patch v.1 (WIP)

work in progress patch for implementing SetIcon for windows ce.  the approach I took was to just use libpr0n to grab the ico file on disk, decode, then construct a HICON.  I wanted this to work on w32 as well, better testing converge, etc.

couple things that are broken.  i was hoping that each icon in the ico file would be a separate container.  right now, the decoder isn't doing that.  Also, on windows mobile (maybe ce too) the V4 bit map stuff is not supported.  Right now, we are not calling this stuff, but we should #ifdef it out instead of #defining the structures.  Lastly, i am not sure about the dwSize math when implementing CreateDIBitmap (i do not think we such always use 4 as the size to multiply h*w by).  as i said, work in progress.
Assignee: nobody → doug.turner

Comment 2

9 years ago
do you need this ?

can you check to see if chatzilla, when installed in Firefox, has a window icon on Windows CE?

Comment 3

9 years ago
(this is not blocking fennec)
tracking-fennec: 1.0-wm+ → ---


9 years ago
Assignee: doug.turner → nobody
WinCE/Windows Mobile support has been removed from the main build system, Spidermonkey, mobile installer, in-app updater and so on (see bug 614720, bug 554087 and all their dependants). Until such point where MS decide to release a Windows Phone 7 NDK and the decision is made to port to that platform, this is WONTFIX.

Filter bugmail on WinCEMassWONTFIX.
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.