dojo.destroy broken under IE since rev. 25179

[Patrick Ruzand]

it seems dojo.destroy (= dojo._destroyElement) has been broken recently (rev. 25179) under IE: an exception is thrown while evaluating the _destroyContainer variable:

l.130                   if(!_destroyContainer || _destroyDoc != doc){
                               _destroyContainer = doc.createElement("div");
                               _destroyDoc = doc;
                       }

_destroyContainer being undefined, it throws an error under IE.

From what I see, it is due to the now missing _destroyContainer variable initialization to null. (was there previously in 1.6).

[Douglas Hays]

Refs #13065

[Kris Zyp]

(In [25191]) Revert rest of dojo.destroy changes, hopefully fix memory leak, refs #13065, fixes #13072 !strict

[bill]

(In [25195]) Put back unload require, it's needed again. Refs #13065, #13072, #12672, #12790 !strict.