commit 5ebb5023b6abb5827ed7e7d9d78374ff2e31114f
author brettw <brettw@chromium.org> Thu Sep 29 14:57:12 2016 -0700
committer Commit bot <commit-bot@chromium.org> Thu Sep 29 21:59:55 2016 +0000
tree c053ef865ddbf86cf442381242d093cce9734e8d
parent f593b8f6c888d2c5c915294c3c4491d65f5377f6

Fix GN issues identified by MSVC static analysis.

MSVC's /analyze identified two missing null checks because it noted that a
pointer was null checked in one place but not another. After some study I
determined that null checks are not necessary for these values so I removed
them.

Although isxdigit takes an int, the input must be an unsigned char or the
behavior is undefined. I added the cast. I think this API is suboptimal.

BUG=643842,427616

Review-Url: https://codereview.chromium.org/2384493003
Cr-Original-Commit-Position: refs/heads/master@{#421945}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: e23e7021f9a06d621088bc13856db9ba93c328c0