Closed Bug 655493 Opened 10 years ago Closed 10 years ago

Remove WinCE code from layout

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Platform:
All
Windows CE
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla6

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

Remove WinCE code from layout/base/tests/TestPoisonArea.cpp
1.57 KB, patch
roc
: review+
Details | Diff | Splinter Review 
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 layout/* parts of the removal:
http://mxr.mozilla.org/mozilla-central/search?string=wince&find=/layout/
The only other remaining WinCE reference in layout is here:
http://mxr.mozilla.org/mozilla-central/source/layout/style/nsCSSScanner.cpp#1309

Are there any other platforms that need pow() rather than powf()? If not, would it be desirable to change this to powf() now?
Attachment #530843 - Flags: review?(roc)
Not exactly a risky patch, but I was submitting a try run anyway:
http://dev.philringnalda.com/tbpl/?tree=Try&rev=f88432556f2e
Comment on attachment 530843 [details] [diff] [review]
Remove WinCE code from layout/base/tests/TestPoisonArea.cpp

Review of attachment 530843 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #530843 - Flags: review?(roc) → review+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/052bbd46f96f
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.