This change fixes a few issues that SonarQube surfaced:
- In a couple places, we were iterating over object fields without testing
`hasOwnProperty`.
- The calls to `testCloseDescription` were passing too many arguments.
- We were testing whether `indexOf` returned a value greater than zero, but
zero is a valid return value.
Reviewed at https://reviews.reviewboard.org/r/8733/
