Fix two minor bugs found by /analyze
BUG=427616
/analyze pointed out that the loop range-check used the comma operator
when && was probably intended. The bug was harmless, but confusing, so
I fixed it.
/analyze also pointed out that the code was checking data read by a
function and then checking whether the read succeeded. Reversing the
order of the checks has no effect except to get rid of the undefined
behavior.
Review URL: https://codereview.chromium.org/678193003
Cr-Commit-Position: refs/heads/master@{#301691}
1 file changed