Change GPUError to a base class instead of a union - #2853

SHA: a7f36ce
Reason: push, by @kainino0x

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

SHA: a7f36ce
Reason: push, by @kainino0x

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

The spec updated in gpuweb/gpuweb#2853 to change
the error types from a union to extended interfaces from a GPUError base
class. IDL slightly differs from current spec text to fix an issue with
the way the constructors were declared.

Bug: 1326472
Change-Id: I83d1d840ef4b57b25980cb8d0da35dea3a0ead82
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3649688
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Commit-Queue: Brandon Jones <bajones@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1004905}

This reverts commit e42a232.

Reason for revert: Looks like this broke compile: https://ci.chromium.org/ui/p/chromium/builders/ci/linux-archive-rel/67275/overview

Original change's description:
> Update GPUError interface hierarchy
>
> The spec updated in gpuweb/gpuweb#2853 to change
> the error types from a union to extended interfaces from a GPUError base
> class. IDL slightly differs from current spec text to fix an issue with
> the way the constructors were declared.
>
> Bug: 1326472
> Change-Id: I83d1d840ef4b57b25980cb8d0da35dea3a0ead82
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3649688
> Reviewed-by: Corentin Wallez <cwallez@chromium.org>
> Reviewed-by: Kai Ninomiya <kainino@chromium.org>
> Commit-Queue: Brandon Jones <bajones@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1004905}

Bug: 1326472
Change-Id: Ib2f2c427c99fcad09f4f9d201cc224fa7a148486
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3654479
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Scott Little <sclittle@chromium.org>
Owners-Override: Scott Little <sclittle@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1004923}

The spec updated in gpuweb/gpuweb#2853 to change
the error types from a union to extended interfaces from a GPUError base
class. IDL slightly differs from current spec text to fix an issue with
the way the constructors were declared.

This CL removes the old union include and gn entries, which caused a
compilation failure on the tree with the last attempt.

Bug: 1326472
Change-Id: I4001096f698eb75eea7cbd570a4640ec1182562b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3654655
Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Commit-Queue: Brandon Jones <bajones@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1005019}

* Change GPUError to a base class instead of a union

* Apply suggestions from code review

Co-authored-by: Brandon Jones <tojiro@gmail.com>

Co-authored-by: Brandon Jones <tojiro@gmail.com>

…eb#2853

WebGPU spec still not great, does not have a general error code. See also gpuweb/gpuweb#2750 and gpuweb/gpuweb#1884 .