tree 17e3e0e2fe406c3c4ceb7c0e8769843198a159f5
parent 8aa4bd065d72815dc335a8317fedec2bc6b26cd0
author dino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc> 1571358573 +0000
committer dino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc> 1571358573 +0000

ctx.font = "" asserts in CSS parser
https://bugs.webkit.org/show_bug.cgi?id=203127
<rdar://problem/56391016>

Reviewed by Devin Rousso.

Source/WebCore:

The HTML specification says:
"values that cannot be parsed as CSS font values are ignored", so
return early if we get an empty string, otherwise the CSS parser
will assert. This was the only case I could find where we sidestepped
most of the parsing infrastructure and injected a raw string.

Test: http/wpt/2dcontext/text-styles/2d.text.font.parse.invalid.html

* html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::setFont):

LayoutTests:

Copy the updated test from my WPT pull request. Once this
is merged into WPT, we won't need this version.

* http/wpt/2dcontext/text-styles/2d.text.font.parse.invalid-expected.txt: Added.
* http/wpt/2dcontext/text-styles/2d.text.font.parse.invalid.html: Added.
* http/wpt/resources/canvas-tests.css: Added.
* http/wpt/resources/canvas-tests.js: Added.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@251270 268f45cc-cd09-0410-ab3c-d52691b4dbfc