Bug 191515

Summary: [css-grid] Import additional grid layout tests and update TestExpectations
Product: WebKit Reporter: Javier Fernandez <jfernandez>
Component: Layout and RenderingAssignee: Javier Fernandez <jfernandez>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, dino, ews, jfernandez, rego, rniwa, simon.fraser, svillar, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews103 for mac-sierra
none
Archive of layout-test-results from ews105 for mac-sierra-wk2
none
Archive of layout-test-results from ews121 for ios-simulator-wk2
none
Archive of layout-test-results from ews116 for mac-sierra
none
Patch none

Description Javier Fernandez 2018-11-11 15:31:27 PST
Adding new layout tests for code already landed in trunk but not properly covered by enough test cases.

Additionally, we should update the TestExpectations with the proper bugs that cause such expected failures.
Comment 1 Javier Fernandez 2018-11-11 15:34:58 PST
Created attachment 354511 [details]
Patch
Comment 2 Build Bot 2018-11-11 16:38:59 PST
Comment on attachment 354511 [details]
Patch

Attachment 354511 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9952220

New failing tests:
fast/css-grid-layout/column-property-should-not-apply-on-grid-container.html
Comment 3 Build Bot 2018-11-11 16:39:01 PST
Created attachment 354515 [details]
Archive of layout-test-results from ews103 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 4 Build Bot 2018-11-11 16:48:35 PST
Comment on attachment 354511 [details]
Patch

Attachment 354511 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/9952239

New failing tests:
fast/css-grid-layout/column-property-should-not-apply-on-grid-container.html
Comment 5 Build Bot 2018-11-11 16:48:37 PST
Created attachment 354516 [details]
Archive of layout-test-results from ews105 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 6 Build Bot 2018-11-11 17:23:22 PST
Comment on attachment 354511 [details]
Patch

Attachment 354511 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/9952273

New failing tests:
fast/css-grid-layout/column-property-should-not-apply-on-grid-container.html
fast/css-grid-layout/preferred-width-computed-after-layout.html
Comment 7 Build Bot 2018-11-11 17:23:24 PST
Created attachment 354519 [details]
Archive of layout-test-results from ews121 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6
Comment 8 Build Bot 2018-11-11 17:34:30 PST
Comment on attachment 354511 [details]
Patch

Attachment 354511 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9952284

New failing tests:
fast/css-grid-layout/column-property-should-not-apply-on-grid-container.html
Comment 9 Build Bot 2018-11-11 17:34:32 PST
Created attachment 354520 [details]
Archive of layout-test-results from ews116 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 10 Javier Fernandez 2018-11-12 04:24:37 PST
Created attachment 354545 [details]
Patch
Comment 11 Dean Jackson 2018-11-12 10:00:17 PST
Comment on attachment 354545 [details]
Patch

rs=me
Comment 12 WebKit Commit Bot 2018-11-12 13:25:15 PST
Comment on attachment 354545 [details]
Patch

Clearing flags on attachment: 354545

Committed r238105: <https://trac.webkit.org/changeset/238105>
Comment 13 WebKit Commit Bot 2018-11-12 13:25:17 PST
All reviewed patches have been landed.  Closing bug.
Comment 14 Radar WebKit Bug Importer 2018-11-12 13:26:27 PST
<rdar://problem/46001156>