Bug 1826672 [wpt PR 39401] - Revert "Nested calc() expressions need to consume all...
commit494cf9481d920eb5f3c2bfd371e84f980ea3881d
authorXiaocheng Hu <xiaochengh@chromium.org>
Thu, 13 Apr 2023 04:33:03 +0000 (13 04:33 +0000)
committermoz-wptsync-bot <wptsync@mozilla.com>
Fri, 14 Apr 2023 11:16:02 +0000 (14 11:16 +0000)
treec5d7b85d0bd82b678f278007f24ccea069b9b734
parent33b3036f8492cc01d3a0a98d53118456120ce73d
Bug 1826672 [wpt PR 39401] - Revert "Nested calc() expressions need to consume all their tokens", a=testonly

Automatic update from web-platform-tests
Revert "Nested calc() expressions need to consume all their tokens"

This reverts commit 5a426f9babceea8ffa93d870690634371a79457b.

Reason for revert: Caused crbug.com/1430953

Original change's description:
> Nested calc() expressions need to consume all their tokens
>
> When parsing a nested calc() expression that had a valid first part
> (term) that was followed by an invalid operator the parsed part was
> returned to the caller - which had already consumed the tokens within
> the sub-expression. The caller only checked that an expression node was
> returned, which meant that the sub-expression was considered valid.
>
> Require that all tokens that are part of the sub-expression have been
> consumed after parsing has completed to fail the parsing in such a case.
>
> Fixed: 1430413
> Change-Id: I3072e22e2a7b81296510173266814a18fb6bfc37
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4400956
> Auto-Submit: Fredrik Söderquist <fs@opera.com>
> Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1126635}

Bug: 1430953,1430413
Change-Id: I866b130615c4328bd4b99716e74e3a045e0f348c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4404759
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1126878}

--

wpt-commits: 50b01fd11e6a8a704b0524de6136ca8e374b3103
wpt-pr: 39401
testing/web-platform/tests/css/css-values/calc-invalid-parsing.html [deleted file]