Remove error-recovery TODO comments from bitcode parser.

Remove error-recovery TODO comments from bitcode parser.

The bitcode parser contained a number of TODO commments about
removing error recovery once the parser was implemented. This
was based on the fact that (1) pnacl-sz in the browser didn't
need it, and (2) it would be trivial to remove.

The advantage of leaving error recovery in is that multiple errors can
be found at once.

It turns out it isn't so easy to remove, since several methods assume
that recovery can be applied, and hence do not need propagate back up
an (optional) error code. Parsing backs out relatively quickly anyway,
since the code exits between bitcode records anyway.

BUG=None
R=stichnot@chromium.org

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=83ccadcf9ee6408bde0dbfe28a764de813a7bf31

[Karl, 2015-08-10 19:17:49]

kschimpf@google.com changed reviewers:
+ dschuff@chromium.org, stichnot@chromium.org

[Jim Stichnoth, 2015-08-12 01:33:01]

lgtm

[Karl, 2015-08-12 17:39:19]

Committed patchset #1 (id:1) manually as
83ccadcf9ee6408bde0dbfe28a764de813a7bf31 (presubmit successful).