Bug 179819

Summary: Move destroyLeftoverChildren call to RenderObject::destroy
Product: WebKit Reporter: Antti Koivisto <koivisto>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, ews, rniwa, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch
zalan: review+, ews: commit-queue-
Archive of layout-test-results from ews107 for mac-elcapitan-wk2
none
patch
none
patch
none
patch none

Description Antti Koivisto 2017-11-17 07:15:52 PST
For safety.
Comment 1 Antti Koivisto 2017-11-17 07:17:44 PST
Created attachment 327169 [details]
patch
Comment 2 zalan 2017-11-17 07:23:32 PST
Comment on attachment 327169 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=327169&action=review

> Source/WebCore/ChangeLog:3
> +        Add release assert that RenderLayer is out of tree before destruction

..is detached before destruction?
Comment 3 Build Bot 2017-11-17 08:11:29 PST
Comment on attachment 327169 [details]
patch

Attachment 327169 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/5274607

Number of test failures exceeded the failure limit.
Comment 4 Build Bot 2017-11-17 08:11:30 PST
Created attachment 327173 [details]
Archive of layout-test-results from ews107 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 5 Antti Koivisto 2017-11-17 08:33:50 PST
Created attachment 327175 [details]
patch
Comment 6 Antti Koivisto 2017-11-17 09:22:27 PST
Created attachment 327184 [details]
patch
Comment 7 Antti Koivisto 2017-11-17 09:57:29 PST
Created attachment 327188 [details]
patch
Comment 8 Radar WebKit Bug Importer 2017-11-17 10:40:30 PST
<rdar://problem/35618234>
Comment 9 WebKit Commit Bot 2017-11-17 11:34:36 PST
Comment on attachment 327188 [details]
patch

Clearing flags on attachment: 327188

Committed r224976: <https://trac.webkit.org/changeset/224976>
Comment 10 WebKit Commit Bot 2017-11-17 11:34:38 PST
All reviewed patches have been landed.  Closing bug.