Bug 77596 - update CSS3 calc() color tests to use pre/post js
Summary: update CSS3 calc() color tests to use pre/post js
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mike Lawther
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-01 20:20 PST by Mike Lawther
Modified: 2012-02-09 07:54 PST (History)
3 users (show)

See Also:


Attachments
Patch (11.14 KB, patch)
2012-02-01 20:22 PST, Mike Lawther
no flags Details | Formatted Diff | Diff
Patch (11.13 KB, patch)
2012-02-01 21:05 PST, Mike Lawther
no flags Details | Formatted Diff | Diff
Patch for landing (11.13 KB, patch)
2012-02-05 22:12 PST, Mike Lawther
no flags Details | Formatted Diff | Diff
Patch for landing (11.88 KB, patch)
2012-02-09 06:25 PST, Mike Lawther
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Lawther 2012-02-01 20:20:09 PST
update CSS3 calc() color tests to use pre/post js
Comment 1 Mike Lawther 2012-02-01 20:22:20 PST
Created attachment 125067 [details]
Patch
Comment 2 Daniel Bates 2012-02-01 20:47:16 PST
Comment on attachment 125067 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=125067&action=review

> LayoutTests/css3/calc/color-hsl.html:5
> +    #simple        { color: hsl(-webkit-calcy(120),-webkit-calc(75%),-webkit-calc(50%)); }

Did you mean to call -webkit-calcy()?

> LayoutTests/css3/calc/color-hsl.html:8
> +    #alpha        { color: hsla(-webkit-calcy(120),-webkit-calc(75%),-webkit-calc(50%), -webkit-calc(0.7)); }

Ditto.

> LayoutTests/css3/calc/color-rgb.html:5
> +    #simple        { color: rgb(-webkit-calcy(10),-webkit-calc(18 * 10),-webkit-calc(30)); }

Ditto.

> LayoutTests/css3/calc/color-rgb.html:7
> +    #percentsimple        { color: rgb(-webkit-calcy(2% + 8%),-webkit-calc(95%),-webkit-calc(3% * 10)); }

Ditto.

> LayoutTests/css3/calc/color-rgb.html:10
> +    #alpha        { color: rgba(-webkit-calcy(10),-webkit-calc(180),-webkit-calc(30), -webkit-calc(0.35 * 2)); }

Ditto.

> LayoutTests/css3/calc/color-rgb.html:12
> +    #percentalpha        { color: rgba(-webkit-calcy(2 * 5%),-webkit-calc(100% - 5%),-webkit-calc(30%),-webkit-calc(0.2 * 3 + 0.1)); }

Ditto.
Comment 3 Kent Tamura 2012-02-01 20:49:15 PST
Comment on attachment 125067 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=125067&action=review

> LayoutTests/css3/calc/color-hsl.html:5
> -#simple        {color: hsl(-webkit-calc(120),-webkit-calc(75%),-webkit-calc(50%));}
> +    #controlsimple { color: hsl(120,75%,50%); }
> +    #simple        { color: hsl(-webkit-calcy(120),-webkit-calc(75%),-webkit-calc(50%)); }

I don't like adding indentation to the top-level code.

> LayoutTests/css3/calc/color-hsl.html:21
>  <script>
> -if (window.layoutTestController)
> -    layoutTestController.dumpAsText();
> -
> -tests = ["simple", "alpha"];
> +    description("Tests that CSS3 calc() can be used with the hsl() and hsla() functions");
> +    
> +    tests = ["simple", "alpha"];

ditto.

> LayoutTests/css3/calc/color-rgb.html:4
> +    #controlsimple { color: rgb(10,180,30); }

ditto.

> LayoutTests/css3/calc/color-rgb.html:29
> +    description("Tests that CSS3 calc() can be used with the rgb() and rgba() functions");

ditto.
Comment 4 Kent Tamura 2012-02-01 20:50:59 PST
Comment on attachment 125067 [details]
Patch

back to r? for Daniel's comment :-)
Comment 5 Mike Lawther 2012-02-01 21:05:40 PST
Created attachment 125072 [details]
Patch
Comment 6 Mike Lawther 2012-02-01 21:07:55 PST
Comment on attachment 125067 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=125067&action=review

>>> LayoutTests/css3/calc/color-hsl.html:5
>>> +    #simple        { color: hsl(-webkit-calcy(120),-webkit-calc(75%),-webkit-calc(50%)); }
>> 
>> Did you mean to call -webkit-calcy()?
> 
> I don't like adding indentation to the top-level code.

Doh! Good catch - fixed.

I've left the indentation as is after a discussion on IRC.

>> LayoutTests/css3/calc/color-hsl.html:8
>> +    #alpha        { color: hsla(-webkit-calcy(120),-webkit-calc(75%),-webkit-calc(50%), -webkit-calc(0.7)); }
> 
> Ditto.

Fixed.

>> LayoutTests/css3/calc/color-hsl.html:21
>> +    tests = ["simple", "alpha"];
> 
> ditto.

Left as per IRC discussion.

>> LayoutTests/css3/calc/color-rgb.html:4
>> +    #controlsimple { color: rgb(10,180,30); }
> 
> ditto.

Left as per IRC discussion.

>> LayoutTests/css3/calc/color-rgb.html:5
>> +    #simple        { color: rgb(-webkit-calcy(10),-webkit-calc(18 * 10),-webkit-calc(30)); }
> 
> Ditto.

Fixed.

>> LayoutTests/css3/calc/color-rgb.html:7
>> +    #percentsimple        { color: rgb(-webkit-calcy(2% + 8%),-webkit-calc(95%),-webkit-calc(3% * 10)); }
> 
> Ditto.

Fixed.

>> LayoutTests/css3/calc/color-rgb.html:10
>> +    #alpha        { color: rgba(-webkit-calcy(10),-webkit-calc(180),-webkit-calc(30), -webkit-calc(0.35 * 2)); }
> 
> Ditto.

Fixed.

>> LayoutTests/css3/calc/color-rgb.html:12
>> +    #percentalpha        { color: rgba(-webkit-calcy(2 * 5%),-webkit-calc(100% - 5%),-webkit-calc(30%),-webkit-calc(0.2 * 3 + 0.1)); }
> 
> Ditto.

Fixed.

>> LayoutTests/css3/calc/color-rgb.html:29
>> +    description("Tests that CSS3 calc() can be used with the rgb() and rgba() functions");
> 
> ditto.

Left as per IRC discussion.
Comment 7 Kent Tamura 2012-02-01 21:10:31 PST
Comment on attachment 125072 [details]
Patch

ok
Comment 8 WebKit Review Bot 2012-02-01 21:44:50 PST
Comment on attachment 125072 [details]
Patch

Attachment 125072 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/11394711

New failing tests:
css3/calc/color-hsl.html
css3/calc/color-rgb.html
Comment 9 Mike Lawther 2012-02-01 22:18:10 PST
The provided link http://queues.webkit.org/results/11394711 doesn't show color-rgb.html or color-hsl.html anywhere. Is this bot flakiness somehow?
Comment 10 Mike Lawther 2012-02-05 22:12:37 PST
Created attachment 125573 [details]
Patch for landing
Comment 11 WebKit Review Bot 2012-02-05 23:20:13 PST
Comment on attachment 125573 [details]
Patch for landing

Rejecting attachment 125573 [details] from commit-queue.

New failing tests:
css3/calc/color-hsl.html
css3/calc/color-rgb.html
Full output: http://queues.webkit.org/results/11434167
Comment 12 Mike Lawther 2012-02-09 06:25:28 PST
Created attachment 126292 [details]
Patch for landing
Comment 13 WebKit Review Bot 2012-02-09 07:54:06 PST
Comment on attachment 126292 [details]
Patch for landing

Clearing flags on attachment: 126292

Committed r107245: <http://trac.webkit.org/changeset/107245>
Comment 14 WebKit Review Bot 2012-02-09 07:54:12 PST
All reviewed patches have been landed.  Closing bug.