Fix up a few bugs in the js code.
Review Request #428 — Created Feb. 24, 2017 and submitted — Latest diff uploaded
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/