Remove WinCE & Windows Mobile code from dom/*

VERIFIED FIXED in mozilla6

Status

()

defect
VERIFIED FIXED
8 years ago
4 months ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

Trunk
mozilla6
All
Windows CE
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

WinCE & Windows Mobile code is being slowly removed by bug 614720 - since it's broken, no longer supported, and only serves to complicate code maintenance. 

This bug is for the dom/* parts of the removal.
Remove WinCE & Windows Mobile ifdefs from dom/*
+ hg rm WinMobileLocationProvider.(h|cpp)

Justin hope you don't mind another r? ; this is much smaller than the patch in the other bug at least! :-)
Attachment #527422 - Flags: review?(dolske)
Passed try (after repeating a few of the tests for intermittent oranges):
http://dev.philringnalda.com/tbpl/?tree=Try&rev=cca063ae0577
Comment on attachment 527422 [details] [diff] [review]
Remove WinCE & Windows Mobile code from dom/*

Bouncing over to jst, looks fine to me though.
Attachment #527422 - Flags: review?(jst)
Attachment #527422 - Flags: review?(dolske)
Attachment #527422 - Flags: review+
Comment on attachment 527422 [details] [diff] [review]
Remove WinCE & Windows Mobile code from dom/*

Review of attachment 527422 [details] [diff] [review]:

r=jst
Attachment #527422 - Flags: review?(jst) → review+
Only change is updated commit message to include the r=jst,dolske.

Carrying forwards r+
Attachment #527422 - Attachment is obsolete: true
Attachment #528969 - Flags: review+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/c39785cf8fb8
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
Build: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0) Gecko/20100101 Firefox/6.0
The changes made in m-c and landed (comment 6) are visible in hg, under beta (i.e. the file WinMobileLocationProvider.cpp is not available anymore).
Setting this as verified for Firefox 6 Beta
Status: RESOLVED → VERIFIED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.