OL, UL and DL should be smartly included in nsHTMLCopyEncoder::IncludeInContext in mozilla/content/base/src/nsDocumentEncoder.cpp. Based on the fact that a parent tag is included only if the child node is completely selected, there must be a generic change for many a tags to be included in context. This will resolve quite a few bugs and specifically bug#62188 can immediately be closed after fixing this.

Lets leave IncludeInContext() out of this (changing summary). If you are saying that ol/ul/dl should be put into the html stream when we have a contained list item going into the stream, that sounds ok. The place to do that is not IncludeInContext() though. It will require some new functionality in the document encoder.

I do not know this code properly but what I mean here is that OL/DL/UL should be included in context or I should say they should be included in the array of ancestors when I select the list partially. There is no problem in cases where list is fully included in the selected range. Based on my requirement for bug# 62188 it should also be included in IncludeInContext().

IncludeInContext() does the following: it chooses what tags will be forced into the html stream if they are anywhere in the parent hierarchy of what is being encoded. That's not what you want. You want to include OL/UL/DL only if one of it's list items is being encoded too. That is a totally different requirement from IncludeInContext() is doing. If you make IncludeInContext() emit OL/UL/DL, then you will not be able to copy a work from inside a list wthout getting terrible results (something like: <ol>foo</ol>).

The days of having a half dozen milestones out in front of us to divide bugs between seem to be gone, though I dont know why. Lumping everything together as far out as I can. I'll pull back things that I am working on as I go.

[ushing these out as far as bugzilla will let me. I'll pull them back as I work on them.