tree 3f6f315081c315d500788f2de9dde5bff9e179fb
parent 59d28c8f2d91d12533cfd0371acbe473b1825967
author Anders Hartvoll Ruud <andruud@chromium.org> 1612267664 -0800
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1612268831 -0800

Don't crash when serializing empty CSSUnparsedValues

The "tokens_" list (which despite its name is really a list of
CSSUnparsedSegments) can be non-empty, but still produce an empty
sequence when tokenized. We're not taking this into account, and can
therefore end up with an empty CSSParserTokenRange, which will DCHECK
when passed to CSSVariableData::Create.

To fix this, move the empty-check until after the CSSParserTokenRange
has been created.

Fixed: 1169941
Change-Id: I3eac2b46dca9999ba5f4b509f8293f604f678198
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2656036
Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#849551}