GitLab

This refactors UnPackArchive to return UnPackStatus instead
of GetLaastError(). Some errors don't have a clear DWORD mapping
and it's more straigthforward to return UnPackStatus as the main result,
and use it to decide success/failure at call sites.

Change-Id: I0aef82ba65a7cd134a2ecd552cee9b600f188e89
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846312Reviewed-by: Sorin Jianu <sorin@chromium.org>
Reviewed-by: Greg Thompson <grt@chromium.org>
Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#709208}