inconsistent API of Map constructor

[bartvde]

When using div as the first parameter of the Map constructor, only a string is accepted. However, when specifying div in the options both string and DOMElement are accepted. The first parameter of the Map constructor should also accept a DOMElement so things are more consistent.

[bartvde]

Comment by Fabian:

No problem, so my "bug" report had a little good side effect. Just to
clarify the map constructor also accepts a DOMelement as the first parameter
its just written wrong ;)
As far as I checked the code in map.js, both variants uses the same code:
  this.div = OpenLayers.Util.getElement(div);
So it just needs to be corrected in the DOC of the constructor settings.

[tschaub]

(In [10170]) Updating docs to indicate that the first argument to the map constructor can be a string or a dom element. (closes #2564)

[tschaub]

This ticket description is a bit misleading. The docs were not complete, but this is not a bug in Map.js.

[bartvde]

Since this is marked as pullup changing milestone to 2.9

[bartvde]

Needs to be open to show up in the pullup query

[bartvde]

Pullup done in r10175