RESOLVED FIXED 183022
[RenderTreeBuilder] Move RenderObject::insertedInto() mutation logic to RenderTreeBuilder
https://bugs.webkit.org/show_bug.cgi?id=183022
Summary [RenderTreeBuilder] Move RenderObject::insertedInto() mutation logic to Rende...
alan
Reported 2018-02-21 15:42:38 PST
ssia.
Attachments
Patch (6.30 KB, patch)
2018-02-21 15:46 PST, alan
no flags
Archive of layout-test-results from ews102 for mac-sierra (3.05 MB, application/zip)
2018-02-21 16:57 PST, EWS Watchlist
no flags
Archive of layout-test-results from ews116 for mac-sierra (3.96 MB, application/zip)
2018-02-21 17:14 PST, EWS Watchlist
no flags
Archive of layout-test-results from ews104 for mac-sierra-wk2 (3.02 MB, application/zip)
2018-02-21 17:20 PST, EWS Watchlist
no flags
Archive of layout-test-results from ews201 for win-future (11.50 MB, application/zip)
2018-02-21 18:12 PST, EWS Watchlist
no flags
Archive of layout-test-results from ews122 for ios-simulator-wk2 (42.60 MB, application/zip)
2018-02-21 18:15 PST, EWS Watchlist
no flags
Archive of layout-test-results from ews200 for win-future (11.62 MB, application/zip)
2018-02-21 20:03 PST, EWS Watchlist
no flags
Archive of layout-test-results from ews124 for ios-simulator-wk2 (42.15 MB, application/zip)
2018-02-21 20:17 PST, EWS Watchlist
no flags
Patch (6.24 KB, patch)
2018-02-21 21:26 PST, alan
no flags
Radar WebKit Bug Importer
Comment 1 2018-02-21 15:43:02 PST
alan
Comment 2 2018-02-21 15:46:26 PST
Antti Koivisto
Comment 3 2018-02-21 15:53:09 PST
Comment on attachment 334421 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=334421&action=review > Source/WebCore/rendering/updating/RenderTreeBuilder.cpp:390 > while (beforeChild && beforeChild->parent() && beforeChild->parent() != &parent) > - beforeChild = beforeChild->parent(); > + beforeChild = beforeChild->parent(); lol
EWS Watchlist
Comment 4 2018-02-21 16:57:49 PST
Comment on attachment 334421 [details] Patch Attachment 334421 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/6615835 New failing tests: imported/blink/fast/multicol/span/spanner-with-relpos-child.html fast/multicol/span/list-multi-column-crash.html imported/blink/fast/multicol/dynamic/relayout-abspos-in-relpos-spanner.html imported/blink/fast/multicol/span/relpos-spanner-with-abspos-child.html fast/multicol/column-span-range-crash.html fast/multicol/float-not-removed-crash.html imported/blink/fast/multicol/span/in-nested-multicol-with-list-item.html imported/blink/fast/multicol/dynamic/remove-content-before-inner-multicol-with-spanner-crash.html fast/multicol/multicol-crazy-nesting.html imported/blink/fast/multicol/dynamic/insert-spanner-after-spanner-with-inner-multicol-with-spanner-crash.html fast/multicol/newmulticol/spanner-crash.html imported/blink/fast/multicol/span/as-inner-multicol-after-composited-layer-crash.html imported/blink/fast/multicol/span/as-inner-multicol.html
EWS Watchlist
Comment 5 2018-02-21 16:57:50 PST
Created attachment 334425 [details] Archive of layout-test-results from ews102 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 6 2018-02-21 17:14:03 PST
Comment on attachment 334421 [details] Patch Attachment 334421 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/6615833 New failing tests: imported/blink/fast/multicol/span/spanner-with-relpos-child.html fast/multicol/span/list-multi-column-crash.html imported/blink/fast/multicol/dynamic/relayout-abspos-in-relpos-spanner.html imported/blink/fast/multicol/span/relpos-spanner-with-abspos-child.html fast/multicol/column-span-range-crash.html fast/multicol/float-not-removed-crash.html imported/blink/fast/multicol/span/in-nested-multicol-with-list-item.html imported/blink/fast/multicol/dynamic/remove-content-before-inner-multicol-with-spanner-crash.html fast/multicol/multicol-crazy-nesting.html imported/blink/fast/multicol/dynamic/insert-spanner-after-spanner-with-inner-multicol-with-spanner-crash.html fast/multicol/newmulticol/spanner-crash.html imported/blink/fast/multicol/span/as-inner-multicol-after-composited-layer-crash.html imported/blink/fast/multicol/span/as-inner-multicol.html
EWS Watchlist
Comment 7 2018-02-21 17:14:04 PST
Created attachment 334427 [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
EWS Watchlist
Comment 8 2018-02-21 17:20:52 PST
Comment on attachment 334421 [details] Patch Attachment 334421 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/6616002 New failing tests: imported/blink/fast/multicol/span/spanner-with-relpos-child.html fast/multicol/span/list-multi-column-crash.html imported/blink/fast/multicol/dynamic/relayout-abspos-in-relpos-spanner.html imported/blink/fast/multicol/span/relpos-spanner-with-abspos-child.html fast/multicol/column-span-range-crash.html fast/multicol/float-not-removed-crash.html imported/blink/fast/multicol/span/in-nested-multicol-with-list-item.html imported/blink/fast/multicol/dynamic/remove-content-before-inner-multicol-with-spanner-crash.html fast/multicol/multicol-crazy-nesting.html imported/blink/fast/multicol/dynamic/insert-spanner-after-spanner-with-inner-multicol-with-spanner-crash.html fast/multicol/newmulticol/spanner-crash.html imported/blink/fast/multicol/span/as-inner-multicol-after-composited-layer-crash.html imported/blink/fast/multicol/span/as-inner-multicol.html
EWS Watchlist
Comment 9 2018-02-21 17:20:54 PST
Created attachment 334428 [details] Archive of layout-test-results from ews104 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 10 2018-02-21 18:12:17 PST
Comment on attachment 334421 [details] Patch Attachment 334421 [details] did not pass win-ews (win): Output: http://webkit-queues.webkit.org/results/6616305 New failing tests: imported/blink/fast/multicol/span/spanner-with-relpos-child.html fast/multicol/span/list-multi-column-crash.html imported/blink/fast/multicol/dynamic/relayout-abspos-in-relpos-spanner.html imported/blink/fast/multicol/span/relpos-spanner-with-abspos-child.html fast/multicol/column-span-range-crash.html fast/multicol/float-not-removed-crash.html imported/blink/fast/multicol/span/in-nested-multicol-with-list-item.html imported/blink/fast/multicol/dynamic/remove-content-before-inner-multicol-with-spanner-crash.html fast/dynamic/continuation-detach-crash.html imported/blink/fast/multicol/dynamic/insert-spanner-after-spanner-with-inner-multicol-with-spanner-crash.html fast/multicol/newmulticol/spanner-crash.html imported/blink/fast/multicol/span/as-inner-multicol-after-composited-layer-crash.html imported/blink/fast/multicol/span/as-inner-multicol.html fast/multicol/multicol-crazy-nesting.html
EWS Watchlist
Comment 11 2018-02-21 18:12:27 PST
Created attachment 334431 [details] Archive of layout-test-results from ews201 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews201 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
EWS Watchlist
Comment 12 2018-02-21 18:15:28 PST
Comment on attachment 334421 [details] Patch Attachment 334421 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/6616096 New failing tests: imported/blink/fast/multicol/span/spanner-with-relpos-child.html fast/multicol/span/list-multi-column-crash.html imported/blink/fast/multicol/dynamic/relayout-abspos-in-relpos-spanner.html imported/blink/fast/multicol/span/relpos-spanner-with-abspos-child.html fast/multicol/column-span-range-crash.html fast/multicol/float-not-removed-crash.html imported/blink/fast/multicol/span/in-nested-multicol-with-list-item.html imported/blink/fast/multicol/dynamic/remove-content-before-inner-multicol-with-spanner-crash.html fast/multicol/multicol-crazy-nesting.html imported/blink/fast/multicol/dynamic/insert-spanner-after-spanner-with-inner-multicol-with-spanner-crash.html fast/multicol/newmulticol/spanner-crash.html imported/blink/fast/multicol/span/as-inner-multicol-after-composited-layer-crash.html imported/blink/fast/multicol/span/as-inner-multicol.html
EWS Watchlist
Comment 13 2018-02-21 18:15:31 PST
Created attachment 334432 [details] Archive of layout-test-results from ews122 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews122 Port: ios-simulator-wk2 Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 14 2018-02-21 20:03:02 PST
Comment on attachment 334421 [details] Patch Attachment 334421 [details] did not pass win-ews (win): Output: http://webkit-queues.webkit.org/results/6616999 New failing tests: imported/blink/fast/multicol/span/spanner-with-relpos-child.html fast/multicol/span/list-multi-column-crash.html imported/blink/fast/multicol/dynamic/relayout-abspos-in-relpos-spanner.html imported/blink/fast/multicol/span/relpos-spanner-with-abspos-child.html fast/multicol/column-span-range-crash.html fast/multicol/float-not-removed-crash.html imported/blink/fast/multicol/span/in-nested-multicol-with-list-item.html imported/blink/fast/multicol/dynamic/remove-content-before-inner-multicol-with-spanner-crash.html fast/dynamic/continuation-detach-crash.html imported/blink/fast/multicol/dynamic/insert-spanner-after-spanner-with-inner-multicol-with-spanner-crash.html fast/multicol/newmulticol/spanner-crash.html imported/blink/fast/multicol/span/as-inner-multicol-after-composited-layer-crash.html imported/blink/fast/multicol/span/as-inner-multicol.html fast/multicol/multicol-crazy-nesting.html
EWS Watchlist
Comment 15 2018-02-21 20:03:12 PST
Created attachment 334434 [details] Archive of layout-test-results from ews200 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews200 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
EWS Watchlist
Comment 16 2018-02-21 20:17:19 PST
Comment on attachment 334421 [details] Patch Attachment 334421 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/6616908 New failing tests: imported/blink/fast/multicol/span/spanner-with-relpos-child.html fast/multicol/span/list-multi-column-crash.html imported/blink/fast/multicol/dynamic/relayout-abspos-in-relpos-spanner.html imported/blink/fast/multicol/span/relpos-spanner-with-abspos-child.html fast/multicol/column-span-range-crash.html fast/multicol/float-not-removed-crash.html imported/blink/fast/multicol/span/in-nested-multicol-with-list-item.html imported/blink/fast/multicol/dynamic/remove-content-before-inner-multicol-with-spanner-crash.html fast/multicol/multicol-crazy-nesting.html imported/blink/fast/multicol/dynamic/insert-spanner-after-spanner-with-inner-multicol-with-spanner-crash.html fast/multicol/newmulticol/spanner-crash.html imported/blink/fast/multicol/span/as-inner-multicol-after-composited-layer-crash.html imported/blink/fast/multicol/span/as-inner-multicol.html
EWS Watchlist
Comment 17 2018-02-21 20:17:21 PST
Created attachment 334435 [details] Archive of layout-test-results from ews124 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews124 Port: ios-simulator-wk2 Platform: Mac OS X 10.12.6
alan
Comment 18 2018-02-21 21:26:23 PST
WebKit Commit Bot
Comment 19 2018-02-21 22:17:50 PST
Comment on attachment 334438 [details] Patch Clearing flags on attachment: 334438 Committed r228914: <https://trac.webkit.org/changeset/228914>
WebKit Commit Bot
Comment 20 2018-02-21 22:17:52 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.