655176 - Remove WinCE code from docshell

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect)

Description

[Ed Morley [:emorley]]

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

This bug is for the /docshell/* & /uriloader/* parts of the removal:

http://mxr.mozilla.org/mozilla-central/search?string=wince&find=/docshell/

http://mxr.mozilla.org/mozilla-central/search?string=wince&find=/uriloader/

Comment 1

[Ed Morley [:emorley]]

In creating the patch for this, I've come across:
> ifeq ($(OS_ARCH),WINNT WINCE)

Here:
http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/Makefile.in#183
and
http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/Makefile.in#208

...which looks broken to me.

However, if I go ahead and remove the WINCE string, that segment of code will now run under WINNT when it didn't before. 

Should I:
a) Remove the segment entirely, since it isn't being run at present.
b) Remove the WINCE string, thereby fixing that segment for WINNT and hope it's still needed?

Those lines were added back in CVS, with CVS blame saying:
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=Makefile.in&branch=1.61&root=/cvsroot&subdir=mozilla/uriloader/exthandler&command=DIFF_FRAMESET&rev1=1.60&rev2=1.61
...however there is no bug number mentioned in the commit message.

Comment 2

[(not currently active) Ted Mielczarek]

Classic. dougt landed a broken fix with r=dougt. :)

Honestly, I would just remove those sections since they haven't done anything useful in the past 6 years and nobody has noticed. The crud about cygwin make is outdated, we no longer support cygwin as a build environment.

Comment 3

[Ed Morley [:emorley]]

Attachment: Remove WinCE code from docshell (incl broken uriloader/exthandler/Makefile.in WINNT |ifeq|s)

Thanks Ted, have removed those sections entirely.

http://dev.philringnalda.com/tbpl/?tree=Try&rev=4a6a7b83227b

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 530602 [details] [diff] [review]
Remove WinCE code from docshell (incl broken uriloader/exthandler/Makefile.in WINNT |ifeq|s)

r=me

Comment 5

[Daniel Holbert [:dholbert]]

http://hg.mozilla.org/mozilla-central/rev/f20bf706d0b5