Bug 125700 - [CSS Shapes] shape-outside animation does not handle 'auto' well
Summary: [CSS Shapes] shape-outside animation does not handle 'auto' well
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Rob Buis
URL:
Keywords:
Depends on:
Blocks: 98664
  Show dependency tree / graph
 
Reported: 2013-12-13 12:48 PST by Rob Buis
Modified: 2014-02-13 03:48 PST (History)
6 users (show)

See Also:

Attachments
Patch (4.30 KB, patch)
2013-12-13 12:53 PST, Rob Buis 		no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2 (453.84 KB, application/zip)
2013-12-13 13:58 PST, Build Bot 		no flags Details
Patch (5.68 KB, patch)
2013-12-13 14:19 PST, Rob Buis 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Rob Buis 2013-12-13 12:48:17 PST 
Right now using auto for shape-outside in a @-webkit-keyframes will crash.
Comment 1 Rob Buis 2013-12-13 12:53:49 PST 
Created attachment 219184 [details]
Patch
Comment 2 Build Bot 2013-12-13 13:58:10 PST 
Comment on attachment 219184 [details]
Patch

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

New failing tests:
fast/shapes/shape-outside-floats/shape-outside-animation.html
Comment 3 Build Bot 2013-12-13 13:58:11 PST 
Created attachment 219195 [details]
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-10  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 4 Rob Buis 2013-12-13 14:19:49 PST 
Created attachment 219197 [details]
Patch
Comment 5 Rob Buis 2013-12-13 14:20:34 PST 
Comment on attachment 219197 [details]
Patch

Now including updated test result.
Comment 6 Rob Buis 2013-12-16 06:53:57 PST 
Landed in r160623.
Comment 7 Csaba Osztrogonác 2014-02-13 03:48:23 PST 
Comment on attachment 219197 [details]
Patch

Cleared review? from attachment 219197 [details] so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).