[webkit-reviews] review granted: [Bug 16808] ASSERT(m_document->frame() == m_view->frame()) failing in CachedPage::clear when running Safari's PLT : [Attachment 18371] patch v2

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 10 12:55:01 PST 2008


Dave Hyatt <hyatt at apple.com> has granted Adam Roben <aroben at apple.com>'s
request for review:
Bug 16808: ASSERT(m_document->frame() == m_view->frame()) failing in
CachedPage::clear when running Safari's PLT
http://bugs.webkit.org/show_bug.cgi?id=16808

Attachment 18371: patch v2
http://bugs.webkit.org/attachment.cgi?id=18371&action=edit

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
Make setWidget explicitly remove the widget rather than patching WidgetWin's
destructor.  This way you do the removal in a cross-platform way and don't have
to patch the other platforms' widget classes.


More information about the webkit-reviews mailing list