Bug 47207

Summary: CSS 2.1 failure: various before-after-* tests fail
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: carol, commit-queue, mrobinson, rniwa, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 47141    
Attachments:
Description Flags
Proposed Patch. Fixes ASSERT.
hyatt: review+, commit-queue: commit-queue-
Proposed Patch. Fixed problem with the expected result of my own test
carol: commit-queue+
Proposed Patch. Rebased to help the queue
commit-queue: commit-queue-
Fix reviewer name in LayoutTests/Changelog. none

Description Simon Fraser (smfr) 2010-10-05 14:02:41 PDT
The following tests fail:

html4/before-after-display-types-001	fail
html4/before-after-floated-001	fail
html4/before-after-images-001	fail
html4/before-after-positioned-001	fail
html4/before-after-table-parts-001	fail
html4/before-after-table-whitespace-001	fail

(these are ref tests).
Comment 1 Carol Szabo 2011-03-11 14:09:46 PST
http://test.csswg.org/suites/css2.1/20101210/html4/before-after-display-types-001.htm causes ASSERT due to the fact that the parent of a CounterRenderer is not an :after or :before renderer. Shall study why and if this is a valid layout shall fix the code.
Comment 2 Carol Szabo 2011-03-11 17:26:09 PST
Have fix for the ASSERT, but test results are still bad. More patches to follow.
Comment 3 Carol Szabo 2011-03-16 13:27:37 PDT
Created attachment 85962 [details]
Proposed Patch. Fixes ASSERT.
Comment 4 Dave Hyatt 2011-03-22 13:43:11 PDT
Comment on attachment 85962 [details]
Proposed Patch. Fixes ASSERT.

r=me
Comment 5 WebKit Commit Bot 2011-03-22 19:49:52 PDT
Comment on attachment 85962 [details]
Proposed Patch. Fixes ASSERT.

Rejecting attachment 85962 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-3', 'build-..." exit_code: 2

Last 500 characters of output:
...........................................................................................................................................................................................................................
fast/css/content ........
fast/css/counters ..
fast/css/counters/complex-before.html -> failed

Exiting early after 1 failures. 6858 tests run.
151.21s total testing time

6857 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
4 test cases (<1%) had stderr output

Full output: http://queues.webkit.org/results/8221665
Comment 6 Carol Szabo 2011-03-22 21:45:48 PDT
Created attachment 86569 [details]
Proposed Patch. Fixed problem with the expected result of my own test
Comment 7 Carol Szabo 2011-03-22 23:10:35 PDT
Created attachment 86573 [details]
Proposed Patch. Rebased to help the queue
Comment 8 WebKit Commit Bot 2011-03-23 00:38:26 PDT
The commit-queue encountered the following flaky tests while processing attachment 86573 [details]:

inspector/debugger/debugger-pause-on-breakpoint.html bug 56903 (authors: pfeldman@chromium.org and podivilov@chromium.org)
The commit-queue is continuing to process your patch.
Comment 9 WebKit Commit Bot 2011-03-23 00:40:35 PDT
Comment on attachment 86573 [details]
Proposed Patch. Rebased to help the queue

Rejecting attachment 86573 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-cq-sf', 'la..." exit_code: 1

Last 500 characters of output:
n/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 81753 = 36be2a6c7a82817f77d2fbc4e21f8a04b22af63f
r81754 = 673be2edca398e0a227ccfc535762ebb3c4d7279
r81755 = 3d6040d948ab4c4f2e1f7120297d1a2229ea7136
r81756 = 45e9d313fc334d71db88c8341dfd13b3203b2043
Done rebuilding .git/svn/refs/remotes/origin/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.

Full output: http://queues.webkit.org/results/8224661
Comment 10 Carol Szabo 2011-03-23 08:05:05 PDT
Created attachment 86617 [details]
Fix reviewer name in LayoutTests/Changelog.
Comment 11 WebKit Commit Bot 2011-03-23 09:39:18 PDT
Comment on attachment 86617 [details]
Fix reviewer name in LayoutTests/Changelog.

Clearing flags on attachment: 86617

Committed r81778: <http://trac.webkit.org/changeset/81778>
Comment 12 WebKit Commit Bot 2011-03-23 09:39:21 PDT
All reviewed patches have been landed.  Closing bug.