Remove WinCE & Windows Mobile code from dom/*

VERIFIED FIXED in mozilla6

Status

()

Core
DOM
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

Trunk
mozilla6
All
Windows CE
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
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.
(Assignee)

Comment 1

6 years ago
Created attachment 527422 [details] [diff] [review]
Remove WinCE & Windows Mobile code from dom/*

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)
(Assignee)

Comment 2

6 years ago
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+
(Assignee)

Comment 5

6 years ago
Created attachment 528969 [details] [diff] [review]
Remove WinCE & Windows Mobile code from dom/*

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+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/c39785cf8fb8
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla6

Comment 7

6 years ago
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
You need to log in before you can comment on or make changes to this bug.