Rearrange PNaCl OnNexeTempFileReply file check to be *before* transferring ownership.

Rearrange PNaCl OnNexeTempFileReply file check to be *before* transferring ownership.

Otherwise, after transferring ownership the old
base_file will always be invalid and we get spurious
error logs, which can be confusing.

This started happening after the refactoring in r274783.
https://codereview.chromium.org/307173002/diff/80001/components/nacl/renderer/pnacl_translation_resource_host.cc

Alternatively, could have checked the file_handle variable that
ends up owning the base_file at the later point, but might as
well check it earlier.

BUG=none
(but it bugged ncbray =))

Committed: https://crrev.com/3ea468bb44663b2bad85c7fd31be6caaf24f7fc4
Cr-Commit-Position: refs/heads/master@{#319170}

[jvoung (off chromium), 2015-03-04 23:04:30]

jvoung@chromium.org changed reviewers:
+ dschuff@chromium.org

[Derek Schuff, 2015-03-04 23:42:40]

lgtm

[jvoung (off chromium), 2015-03-04 23:56:12]

The CQ bit was checked by jvoung@chromium.org

[commit-bot: I haz the power, 2015-03-04 23:57:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/982663002/20001

[commit-bot: I haz the power, 2015-03-05 00:14:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-05 00:15:39]

Patchset 2 (id:??) landed as
https://crrev.com/3ea468bb44663b2bad85c7fd31be6caaf24f7fc4
Cr-Commit-Position: refs/heads/master@{#319170}