Closed
Bug 654472
Opened 14 years ago
Closed 14 years ago
Remove WinCE & Windows Mobile code from embedding/*
Categories
(Core :: General, defect)
Tracking
()
RESOLVED
FIXED
mozilla6
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(1 file, 1 obsolete file)
24.15 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
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 embedding/* parts of the removal:
http://mxr.mozilla.org/mozilla-central/search?string=wince&find=/embedding/
There isn't a Core::Embedding General component, so filing this in Core::General.
Assignee | ||
Comment 1•14 years ago
|
||
Removes WinCE ifdefs from embedding/*, which means:
- MOZ_FLASH_ACTIVEX_PATCH (http://mxr.mozilla.org/mozilla-central/search?string=MOZ_FLASH_ACTIVEX_PATCH)
- USE_HTIFACE (http://mxr.mozilla.org/mozilla-central/search?string=htiface)
...are no longer defined, so can be removed too.
This causes MozActiveX-flash.res to no longer be needed, so have hg removed MozActiveX-flash.rc as well.
Assignee: nobody → bmo
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•14 years ago
|
||
Comment on attachment 529756 [details] [diff] [review]
Remove WinCE code (incl MOZ_FLASH_ACTIVEX_PATCH & USE_HTIFACE) from embedding/*
Passed try: http://dev.philringnalda.com/tbpl/?tree=Try&rev=0e42b10dba46
Attachment #529756 -
Flags: review?(benjamin)
Updated•14 years ago
|
Attachment #529756 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 3•14 years ago
|
||
Only change is updated commit message to include r=bsmedberg
Carrying forwards r+
Attachment #529756 -
Attachment is obsolete: true
Attachment #530659 -
Flags: review+
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 4•14 years ago
|
||
Assignee | ||
Updated•14 years ago
|
Target Milestone: --- → mozilla6
You need to log in
before you can comment on or make changes to this bug.
Description
•