Negative integers in @font-palette-values are invalid
Created attachment 439246 [details] Patch
<rdar://problem/83528806>
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Created attachment 439252 [details] Patch
Comment on attachment 439252 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=439252&action=review > Source/WebCore/css/parser/CSSPropertyParser.cpp:4853 > + return consumeInteger(range, 0); I should change the parsed types to be unsigned, too.
Created attachment 439277 [details] Patch
Comment on attachment 439277 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=439277&action=review > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:468 > + auto number = adoptCF(CFNumberCreate(kCFAllocatorDefault, kCFNumberSInt64Type, &index)); Nope
Created attachment 439397 [details] Patch
Created attachment 439414 [details] Patch
Created attachment 439438 [details] Patch
Committed r283188 (242236@main): <https://commits.webkit.org/242236@main>
Comment on attachment 439438 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=439438&action=review > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:456 > auto light = kCTFontPaletteLight; > - auto number = adoptCF(CFNumberCreate(kCFAllocatorDefault, kCFNumberSInt64Type, &light)); > + auto number = adoptCF(CFNumberCreate(kCFAllocatorDefault, kCFNumberCFIndexType, &light)); I am perhaps the biggest fan of "auto" in the entire WebKit project, but here is a place where it’s just not appropriate. Needs to be: CFIndex light = kCTFontPaletteLight;
Committed r283197 (242244@main): <https://commits.webkit.org/242244@main>
Committed r283540 (242505@main): <https://commits.webkit.org/242505@main>