Bug 126120 - Always tear down render trees, even if Document is not attached().
Summary: Always tear down render trees, even if Document is not attached().
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Andreas Kling
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-21 08:09 PST by Andreas Kling
Modified: 2014-10-20 10:22 PDT (History)
7 users (show)

See Also:

Attachments
Patch (3.18 KB, patch)
2013-12-21 08:09 PST, Andreas Kling 		koivisto: review+
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2 (575.26 KB, application/zip)
2013-12-21 19:21 PST, Build Bot 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.