<Swing Dev> [9] Review request for 8149631: rgb(...) CSS color values are not parsed properly

[Semyon Sadetsky]

Thanks, Sergey. The webrev is corrected.

On 3/14/2016 4:22 PM, Sergey Bylokhov wrote:
> Something strange occurs in the webrev for the testcase, there is no 
> diff.
>
> On 14.03.16 14:29, Semyon Sadetsky wrote:
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8149631
>> webrev: http://cr.openjdk.java.net/~ssadetsky/8149631/webrev.00/
>>
>> The issue is a regression of 4419748 which added support for border
>> sides but it parse the rgb(...) color values wrong. Also before 4419748
>> the parenthesis notations worked only for CSS attribute that is not
>> expected to contain several values. The reason is splitting the
>> multi-value CSS attribute (for example, "background: rgb(0, 0, 0) url(
>> 'b.gif' ) no-repeat;") using space divider didn't take into account that
>> spaces within parenthesis should not be used as top-level dividers.
>> Fixing the value splitting should correct the situation for (...)
>> entries in all multi-value CSS attributes.
>>
>> --Semyon
>
>