[webkit-reviews] review requested: [Bug 16951] Assertion failure in FrameView::scheduleRelayout (!m_frame->document() || !m_frame->document()->inPageCache()) when going back from a page with a focused popup : [Attachment 18570] Do not allow focus to change in pages that are in the back/forward cache

[bugzilla-daemon at webkit.org]

mitz at webkit.org has asked  for review:
Bug 16951: Assertion failure in FrameView::scheduleRelayout
(!m_frame->document() || !m_frame->document()->inPageCache()) when going back
from a page with a focused popup
http://bugs.webkit.org/show_bug.cgi?id=16951

Attachment 18570: Do not allow focus to change in pages that are in the
back/forward cache
http://bugs.webkit.org/attachment.cgi?id=18570&action=edit

------- Additional Comments from mitz at webkit.org
Cannot make a regression test because the back/forward cache is disabled in
DumpRenderTree.

Besides clearing the focused node, -[WebHTMLView resignFirstResponder] also
clears the selection (if it is not in editable content). The patch leaves the
selection clearing behavior intact.