Closed Bug 461849 Opened 13 years ago Closed 12 years ago
icon decoder wince fixes
we removed much of the code in our wince shunt library. This patch allows icon decoder to build.
Assignee: nobody → doug.turner
Comment on attachment 344983 [details] [diff] [review] patch v.1 does it make sense to just not build the icon decoder rather than disabling it?
we could try that too!
disabled icon decoder.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
pav, review ping? If we're not going to fix icon decoder, can you explain why on the bug?
we simply disabled the icon decoder via configure. we could enable it, but not sure why we need the support on windows mobile.
Is this the cause of favicons not being displayed?
Status: RESOLVED → REOPENED
tracking-fennec: --- → ?
Resolution: WONTFIX → ---
same as before, just updated to the tip of m-c. Without this patch, and simply disabling icon decoder support via configure, we do not display favicons.
Comment on attachment 362916 [details] [diff] [review] patch v.2 please add some comments as to why we can't implement those functions.
Attachment #362916 - Flags: review?(pavlov) → review+
marking fixed. needs to land on 1.9.1
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Comment on attachment 362916 [details] [diff] [review] patch v.2 a191=beltzner
Attachment #362916 - Flags: approval1.9.1? → approval1.9.1+
I can view favicons on HTC Touch. verified FIXED on build: Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.3a1pre) Gecko/20090818 Fennec/1.0a3pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.