SWEEP: Replace IndexOutOfRangeException with ArgumentOutOfRangeException except for indexer use, #1023

[paulirwin]

• You've read the Contributor Guide and Code of Conduct.
• You've included unit or integration tests for your change, where applicable.
• You've included inline docs for your change, where applicable.
• There's an open issue for the PR that you are making. If you'd like to propose a change, please open an issue to discuss the change or find an existing issue.

Replaces IndexOutOfRangeException use with ArgumentOutOfRangeException when thrown from a method rather than an indexer.

Fixes #1023

Description

It is not correct in .NET to use IndexOutOfRangeException to throw exceptions from a non-indexer method when an argument is out of range, even if that argument is named index; that is what ArgumentOutOfRangeException is for. Additionally, this has the added benefit of having the paramName set to know which argument is out of range.