Bug 1645429 - Use std::reverse_iterator instead of mozilla::ReverseIterator where possible. r=froydnj

mozilla::ReverseIterator is required for "stashing iterators", but otherwise
std::reverse_iterator can be used and is potentially more efficient.

Also fix some other glitches in iterator type definitions.

Differential Revision: https://phabricator.services.mozilla.com/D79489

Also adds missing includes in some files, these were previously only transivitely
included through mozilla/TypeTraits.h.

Differential Revision: https://phabricator.services.mozilla.com/D68561

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D68371

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D68369

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D68368

--HG--
extra : moz-landing-system : lando

Bug 1549694 - Don't use check as an identifier in IntegerRange.h, since it conflicts with system headers in Mac and Windows. r=froydnj

Differential Revision: https://phabricator.services.mozilla.com/D30183

--HG--
extra : moz-landing-system : lando

# ignore-this-changeset

--HG--
extra : amend_source : 4d301d3b0b8711c4692392aa76088ba7fd7d1022

MakeRange is just way too generic for this sort of thing.

--HG--
extra : source : c1d3256e987c25935cc8bd321c413a1e0c7586b3

--HG--
extra : source : ca970199a7f1ee9adc11019a65ef8eba1f2a491e

Bug 1175485 part 1 - Allow inner iterator of ReverseIterator deref to any type, and change IntegerIterator, EnumeratedRange, and nsFrameList::Iterator to return value type instead of a reference. r=roc,waldo

--HG--
extra : source : 1e932a13595a1d3862f171e7e7435422d890fbf0

--HG--
extra : source : 2977cf3c5b545826a4ec0cdf245e804574cf7ff9