Closed Bug 377410 Opened 13 years ago Closed 13 years ago
MFCEmbed is very poor code that doesn't use good embedding APIs or practices. I'd like to remove it, since people keep trying to base real products off of it.
This and removing the actual files.
Attachment #261479 - Flags: review?(dougt)
Comment on attachment 261479 [details] [diff] [review] Remove mfcembed, rev. 1 we never really needed another embedding example. Sorry Chak.
Attachment #261479 - Flags: review?(dougt) → review+
fixed on trunk
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
This change breaks this link, what is the best example of this now? http://developer.mozilla.org/en/docs/Mozilla_Embedding_FAQ:How_do_I...#How_do_I_get_popup_windows_working.3F
Sorry to be clearer it breaks the link given in the embedding FAQ. http://lxr.mozilla.org/mozilla/source/embedding/tests/mfcembed/MfcEmbed.cpp#688
Well, that example no longer exists. Epiphany and other gecko-based browsers implement this solution, however.
Benjamin, should this checkin also have removed references to the mfcembed makefiles from allmakefiles.sh?
You need to log in before you can comment on or make changes to this bug.