Closed Bug 503573 Opened 15 years ago Closed 15 years ago

moz-icon doesn't work on Window CE

Categories

(Core :: Widget: Win32, defect)

ARM
Windows CE
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: zpao, Assigned: Dolske)

References

Details

(Whiteboard: [nv])

Attachments

(1 file, 1 obsolete file)

moz-icon:// URIs don't render icons like they're supposed to on Windows CE.

These are used in the download manager & when browsing file:// URIs.

From what I understand, this is likely a limitation of the Windows CE API, but we'll need to look into it.
Whiteboard: [nv]
This doesn't look *too* bad... Most of the code that's #ifndef WINCE in modules/libpr0n/decoders/icon/win/nsIconChannel.cpp will get going with a few small tweaks. From a bit of experimentation, the two main tasks seem to be:

* Implement a replacement for ::GetDIBits(), which isn't available on WinCE
* Avoid usage of nsIFile.normalize(), or implement that on WinCE.
Assignee: nobody → dolske
Depends on: 506811
Attached patch Patch v.1 (obsolete) — Splinter Review
Yay, now working. Tested with icons in the download manager, and by viewing file:///Hard Disk or file:///Windows
Attachment #390985 - Flags: review?(vladimir)
Attached patch Patch v.2Splinter Review
Oops. Forgot to clean up stuff in the first patch.
Attachment #390985 - Attachment is obsolete: true
Pushed http://hg.mozilla.org/mozilla-central/rev/95aca25829be
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: