[css-grid] CRASH when getting the computed style of a grid with only absolutely positioned children
Created attachment 280823 [details] Patch
Comment on attachment 280823 [details] Patch Attachment 280823 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/1468378 New failing tests: fast/css-grid-layout/grid-template-shorthand-get-set.html
Created attachment 280827 [details] Archive of layout-test-results from ews100 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-yosemite Platform: Mac OS X 10.10.5
Comment on attachment 280823 [details] Patch Attachment 280823 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/1468383 New failing tests: fast/css-grid-layout/grid-template-shorthand-get-set.html
Created attachment 280830 [details] Archive of layout-test-results from ews106 for mac-yosemite-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews106 Port: mac-yosemite-wk2 Platform: Mac OS X 10.10.5
Comment on attachment 280823 [details] Patch Attachment 280823 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/1468381 New failing tests: fast/css-grid-layout/grid-template-shorthand-get-set.html
Created attachment 280831 [details] Archive of layout-test-results from ews126 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews126 Port: ios-simulator-wk2 Platform: Mac OS X 10.11.4
Comment on attachment 280823 [details] Patch Attachment 280823 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/1468396 New failing tests: editing/selection/selection-in-iframe-removed-crash.html fast/css-grid-layout/grid-template-shorthand-get-set.html
Created attachment 280833 [details] Archive of layout-test-results from ews117 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews117 Port: mac-yosemite Platform: Mac OS X 10.10.5
Created attachment 280911 [details] Patch adjusted test results
Comment on attachment 280911 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=280911&action=review > Source/WebCore/css/CSSComputedStyleDeclaration.cpp:1129 > + auto& positions = isRowAxis ? downcast<RenderGrid>(renderer)->columnPositions() : downcast<RenderGrid>(renderer)->rowPositions(); Even though it’s only used in one line of code, the local variable is a big help here, so I’d do it: auto& grid = downcast<RenderGrid>(*renderer); auto& positions = isRowAxis ? grid.columnPositions() : grid.rowPositions();
Committed r201919: <http://trac.webkit.org/changeset/201919>