On Tue, 13 Oct 2026 15:48:10 GMT, Andy Goryachev <[email protected]> wrote:

>> Yeah this is also my understanding.
>
> Before this PR the error propagated to CssParser.ParserError.PropertySetError 
> which according to the speec "Encapsulates errors arising from applying a 
> style to a Node."  (L5344)
> 
> so we probably should handle this case (and maybe add a test?)

Same as the other answer, `resetToInitialValues` had no `catch` block at all 
before, also also was not adding any errors. Again a preexisting inconsistency, 
which we again could change or not. Any adjustment will also change the other 
case. Same applies to my previous comment.

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/2343#discussion_r4261326320

Reply via email to