Closed Bug 409300 Opened 17 years ago Closed 16 years ago

Window Mobile Support on trunk

Categories

(Core :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: dougt, Unassigned)

Details

(Keywords: mobile)

Attachments

(11 files)

 
not terribly happy with the MingW path mangling change.
Attachment #294120 - Flags: review?
some of the types in txArgumentType are already defined in the Windows Mobile SDK.  We prefixed TX in front of the defines.

also, maybe we should do the same for txEXSLTFunctionCall eType's instead of undef DIFFERENCE?
Attachment #294121 - Flags: review?
Attached patch js patchSplinter Review
GetLocalTime is already defined on windows mobile.

also max isn't defined on windows mobile; maybe there is a better place for this?
Attachment #294123 - Flags: review?
Attached patch lcms patchSplinter Review
Attachment #294126 - Flags: review?
Attached file imglib
moving pngrtran from the encoder list to the decoder list.  is this correct?
Attachment #294127 - Flags: review?
Attached patch network patchSplinter Review
disable autodial on windows mobile.
Attachment #294128 - Flags: review?(dougt)
Attached patch nspr patchSplinter Review
nsprpub/pr/include/md/_wince.cfg
nsprpub/pr/include/md/_wince.h

are the same as on the NSPRPUB_WINCE_04122006_BRANCH branch.
Attachment #294130 - Flags: review?
Attached patch toolkit patchSplinter Review
no nsDownloadScanner scanner on wince.
Attachment #294131 - Flags: review?
maybe we should use GetProcAddress@ and pass a wide string?  benjamin, i heard that usage of ASCII are now bugs?
Attachment #294132 - Flags: review?
Attached patch widget patchSplinter Review
mostly scoping issues.  patch at nsWindow.cpp ~line 2040 maybe a fix for another bug.
Attachment #294133 - Flags: review?
Keywords: mobile
these patches were by atotic.
Attachment #294137 - Attachment is patch: true
Attachment #294137 - Attachment mime type: application/octet-stream → text/plain
Presumably you meant to provide requestees for these requests?
Comment on attachment 294133 [details] [diff] [review]
widget patch

clearing flag.
Attachment #294133 - Flags: review?
Comment on attachment 294132 [details] [diff] [review]
unicode / *A function.

clearing flag.
Attachment #294132 - Flags: review?
Comment on attachment 294131 [details] [diff] [review]
toolkit patch

clearing flag.
Attachment #294131 - Flags: review?
Comment on attachment 294128 [details] [diff] [review]
network patch

clearing flag.
Attachment #294128 - Flags: review?(dougt)
Comment on attachment 294127 [details]
imglib

clearing flag.
Attachment #294127 - Flags: review?
Comment on attachment 294126 [details] [diff] [review]
lcms patch

clearing flag.
Attachment #294126 - Flags: review?
Comment on attachment 294123 [details] [diff] [review]
js patch

clearing flag.
Attachment #294123 - Flags: review?
Comment on attachment 294121 [details] [diff] [review]
content xslt patch

clearing flag.
Attachment #294121 - Flags: review?
Comment on attachment 294120 [details] [diff] [review]
buildconfig patch

clearing flag.
Attachment #294120 - Flags: review?
Comment on attachment 294130 [details] [diff] [review]
nspr patch

clearing flag.
Attachment #294130 - Flags: review?
I filed separate bugs for each of the issues.  It seemed easier to ask for reviews this way.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Resolution: FIXED → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: