Bug 126120

Summary: Always tear down render trees, even if Document is not attached().
Product: WebKit Reporter: Andreas Kling <kling>
Component: Layout and RenderingAssignee: Andreas Kling <kling>
Status: RESOLVED INVALID    
Severity: Normal CC: buildbot, commit-queue, esprehn+autocc, japhet, kangil.han, kling, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
koivisto: review+, buildbot: commit-queue-
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2 none

Description Andreas Kling 2013-12-21 08:09:05 PST 
Always tear down render trees, even if Document is not attached().
Comment 1 Andreas Kling 2013-12-21 08:09:33 PST 
Created attachment 219852 [details]
Patch
Comment 2 Build Bot 2013-12-21 19:21:48 PST 
Comment on attachment 219852 [details]
Patch

Attachment 219852 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/45128046

New failing tests:
plugins/return-error-from-new-stream-doesnt-invoke-destroy-stream.html
userscripts/user-script-top-frame-only.html
plugins/plugin-document-back-forward.html
fast/frames/iframe-remove-after-id-change.html
fast/loader/reload-zero-byte-plugin.html
plugins/inner-html-display-none.html
Comment 3 Build Bot 2013-12-21 19:21:50 PST 
Created attachment 219859 [details]
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-10  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 4 Andreas Kling 2014-10-20 10:22:19 PDT 
This change doesn't make sense anymore.