Bug 71900

Summary: CSS Exclusions: parse the wrap-margin and wrap-padding properties
Product: WebKit Reporter: Alexandru Chiculita <achicu>
Component: CSSAssignee: Mihnea Ovidenie <mihnea>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dglazkov, macpherson, mihnea, simon.fraser, webkit-bug-importer, webkit.review.bot, xqb748
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
URL: http://dev.w3.org/csswg/css3-exclusions/#wrap-margin-property
Bug Depends on:    
Bug Blocks: 57311    
Attachments:
Description Flags
Patch
webkit.review.bot: commit-queue-
Patch 2
webkit.review.bot: commit-queue-
Patch 3
none
Patch for landing none

Description Alexandru Chiculita 2011-11-09 06:33:57 PST
Add parsing support for wrap-margin and wrap-padding of the CSS Exclusions spec.

http://dev.w3.org/csswg/css3-exclusions/#wrap-margin-property
http://dev.w3.org/csswg/css3-exclusions/#wrap-padding-property
Comment 1 Radar WebKit Bug Importer 2011-11-09 09:25:37 PST
<rdar://problem/10419706>
Comment 2 Mihnea Ovidenie 2011-11-14 00:00:20 PST
Created attachment 114892 [details]
Patch
Comment 3 WebKit Review Bot 2011-11-14 00:52:41 PST
Comment on attachment 114892 [details]
Patch

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

New failing tests:
svg/css/getComputedStyle-basic.xhtml
fast/css/getComputedStyle/computed-style.html
fast/css/getComputedStyle/computed-style-without-renderer.html
Comment 4 Mihnea Ovidenie 2011-11-14 01:45:05 PST
Created attachment 114900 [details]
Patch 2

Now including the missed tests.
Comment 5 WebKit Review Bot 2011-11-14 05:44:32 PST
Comment on attachment 114900 [details]
Patch 2

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

New failing tests:
svg/css/getComputedStyle-basic.xhtml
fast/css/getComputedStyle/computed-style.html
fast/css/getComputedStyle/computed-style-without-renderer.html
Comment 6 Mihnea Ovidenie 2011-11-14 07:38:07 PST
(In reply to comment #5)
> (From update of attachment 114900 [details])
> Attachment 114900 [details] did not pass chromium-ews (chromium-xvfb):
> Output: http://queues.webkit.org/results/10472225
> 
> New failing tests:
> svg/css/getComputedStyle-basic.xhtml
> fast/css/getComputedStyle/computed-style.html
> fast/css/getComputedStyle/computed-style-without-renderer.html

Not sure why it failed this time since i have modified the expected results.
Comment 7 Adam Barth 2011-11-14 08:55:24 PST
> Not sure why it failed this time since i have modified the expected results.

Perhaps different ports have different expectations for these tests based on which features they have enabled?
Comment 8 Mihnea Ovidenie 2011-11-14 09:29:33 PST
(In reply to comment #7)
> > Not sure why it failed this time since i have modified the expected results.
> 
> Perhaps different ports have different expectations for these tests based on which features they have enabled?

Actually, the failure comes from the chromium bot. Is there a way to see why a certain test has failed and the diff associated with the failure?
Comment 9 Mihnea Ovidenie 2011-11-14 23:20:21 PST
Created attachment 115103 [details]
Patch 3

Same patch with expected results for chromium-win.
Comment 10 WebKit Review Bot 2011-11-17 00:43:04 PST
Comment on attachment 115103 [details]
Patch 3

Rejecting attachment 115103 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
n/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.
Updating chromium port dependencies using gclient...

________ running '/usr/bin/python tools/clang/scripts/update.py --mac-only' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'

________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
Updating webkit projects from gyp files...

Full output: http://queues.webkit.org/results/10340123
Comment 11 Adam Barth 2011-11-17 00:49:52 PST
1 out of 3 hunks FAILED -- saving rejects to file Source/WebCore/rendering/style/StyleRareNonInheritedData.cpp.rej
Comment 12 Mihnea Ovidenie 2011-11-17 03:24:58 PST
Created attachment 115553 [details]
Patch for landing
Comment 13 WebKit Review Bot 2011-11-17 04:46:34 PST
Comment on attachment 115553 [details]
Patch for landing

Clearing flags on attachment: 115553

Committed r100604: <http://trac.webkit.org/changeset/100604>
Comment 14 WebKit Review Bot 2011-11-17 04:46:40 PST
All reviewed patches have been landed.  Closing bug.