Bug 169134 - Add assertion in Frame::setView() to make sure associated document has no living render tree
Summary: Add assertion in Frame::setView() to make sure associated document has no liv...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-03-03 08:50 PST by Chris Dumez
Modified: 2017-03-03 09:50 PST (History)
3 users (show)

See Also:


Attachments
Patch (1.52 KB, patch)
2017-03-03 08:51 PST, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2017-03-03 08:50:53 PST
Add assertion in Frame::setView() to make sure associated document has no living render tree. This is not supposed to happen and r213311 fixed one case where this could happen. We need to find out if it can still happen.
Comment 1 Chris Dumez 2017-03-03 08:51:55 PST
Created attachment 303322 [details]
Patch
Comment 2 Brent Fulgham 2017-03-03 09:22:22 PST
Comment on attachment 303322 [details]
Patch

Yes! r=me.
Comment 3 WebKit Commit Bot 2017-03-03 09:50:18 PST
Comment on attachment 303322 [details]
Patch

Clearing flags on attachment: 303322

Committed r213369: <http://trac.webkit.org/changeset/213369>
Comment 4 WebKit Commit Bot 2017-03-03 09:50:22 PST
All reviewed patches have been landed.  Closing bug.