Bug 129825 - [CSS Shapes] inset args and radial args should serialize to the simplest form
Summary: [CSS Shapes] inset args and radial args should serialize to the simplest form
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Bear Travis
URL:
Keywords:
Depends on:
Blocks: 98664
  Show dependency treegraph
 
Reported: 2014-03-06 14:08 PST by Rebecca Hauck
Modified: 2014-04-11 10:09 PDT (History)
9 users (show)

See Also:


Attachments
Test case for bug (1.10 KB, text/html)
2014-03-06 14:09 PST, Rebecca Hauck
no flags Details
Patch (41.51 KB, patch)
2014-04-09 17:41 PDT, Bear Travis
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion (490.55 KB, application/zip)
2014-04-09 18:59 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 (547.31 KB, application/zip)
2014-04-09 19:08 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion (544.97 KB, application/zip)
2014-04-09 19:12 PDT, Build Bot
no flags Details
Updated patch (43.57 KB, patch)
2014-04-10 13:31 PDT, Bear Travis
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rebecca Hauck 2014-03-06 14:08:14 PST
From Section 3.3 of the spec:
http://dev.w3.org/csswg/css-shapes/#basic-shape-serialization

To serialize the <basic-shape> functions, serialize as per their individual grammars, in the order the grammars are written in, avoiding calc() expressions where possible, avoiding calc() transformations, omitting components when possible without changing the meaning, joining space-separated tokens with a single space, and following each serialized comma with a single space.

The attached tests has 2 examples of inset() serialization:

----------------------------
value:  inset(10px)  
actual: inset(10px 10px 10px 10px)
expected: inset(10px) 
-----------------------------
value: inset(10px round 20px) 
actual: inset(10px 10px 10px 10px round 20px 20px 20px 20px / 20px 20px 20px 20px)
expected: inset(10px round 20px) 
-----------------------------

Note: This uses testharness.js and the relative path in the test assumes the test is run from LayoutTests/fast/shapes/parsing
Comment 1 Rebecca Hauck 2014-03-06 14:09:39 PST
Created attachment 226037 [details]
Test case for bug
Comment 2 Zoltan Horvath 2014-04-03 08:53:46 PDT
Rob is started to work on this bug in Blink: https://code.google.com/p/chromium/issues/detail?id=350092 Let's wait for him, then we can port the patch over to WebKit.
Comment 3 Bear Travis 2014-04-09 17:41:34 PDT
Created attachment 229004 [details]
Patch
Comment 4 Build Bot 2014-04-09 18:59:43 PDT
Comment on attachment 229004 [details]
Patch

Attachment 229004 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/4906653015080960

New failing tests:
css3/masking/clip-path-animation.html
Comment 5 Build Bot 2014-04-09 18:59:46 PDT
Created attachment 229013 [details]
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-06  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 6 Build Bot 2014-04-09 19:07:59 PDT
Comment on attachment 229004 [details]
Patch

Attachment 229004 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5646063847342080

New failing tests:
css3/masking/clip-path-animation.html
Comment 7 Build Bot 2014-04-09 19:08:05 PDT
Created attachment 229014 [details]
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-12  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 8 Build Bot 2014-04-09 19:11:58 PDT
Comment on attachment 229004 [details]
Patch

Attachment 229004 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/4737813689925632

New failing tests:
css3/masking/clip-path-animation.html
Comment 9 Build Bot 2014-04-09 19:12:04 PDT
Created attachment 229015 [details]
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-04  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 10 Bear Travis 2014-04-10 13:31:28 PDT
Created attachment 229074 [details]
Updated patch
Comment 11 Dirk Schulze 2014-04-11 09:19:13 PDT
Comment on attachment 229074 [details]
Updated patch

r=me
Comment 12 WebKit Commit Bot 2014-04-11 10:09:47 PDT
Comment on attachment 229074 [details]
Updated patch

Clearing flags on attachment: 229074

Committed r167132: <http://trac.webkit.org/changeset/167132>
Comment 13 WebKit Commit Bot 2014-04-11 10:09:54 PDT
All reviewed patches have been landed.  Closing bug.