tree ffad21f2f92ef1fc853f77721b06c27563de2522
parent 28c1fba1c02eb0c8fedb9794645e4a2dc06d1904
author Corentin Wallez <cwallez@chromium.org> 1545044125 +0000
committer Commit Bot service account <commit-bot@chromium.org> 1545044125 +0000

WireServer: don't skip destroy on error objects.

An old-style builder might become an error object when a
validation error occurs on one of its methods. It still has
to be freed, so the correct check for "destroy" is to check
that the handle is not null.

BUG=chromium:914819
BUG=chromium:914867
BUG=chromium:914964
BUG=chromium:915083

Change-Id: I349faffa48f369b72c603d47dd23ffae7d86891e
Reviewed-on: https://dawn-review.googlesource.com/c/3323
Reviewed-by: Stephen White <senorblanco@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>