OrderBy filter doesn't work with dates (momentJS and native dates)

[Toilal]

Since 1.3.7, OrderBy filter isn't working anymore with momentJS or native dates.

I've build a plunker to reproduce the issue : http://plnkr.co/edit/tzcbq7gJgOXvY5gj5QI0?p=preview

Using angular 1.3.6 fix the issue.

[cherbst]

I have tracked down this issue to the following commit: a097aa9.
This happens because the order in which the compared values are tried to be converted to primitives has been changed by this commit. Previously valueOf() has been tried before toString(). With this commit it is the other way around. This leads to dates being compared as strings which is obviously wrong.

[Toilal]

Thanks @cherbst.

It's probably this valueOf() call that was making things works before. Calling valueOf() on native date or MomentJS date returns a timestamp which is comparable.

[pkozlowski-opensource]

Yeh, it sounds like a regression that we allowed to slip as our tests were not catching this exact situation... Sent a PR with a fix (#10516).