<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - REGRESSION(r188659): Non main frame scrollable areas don't work for pages restored from the page cache"
href="https://bugs.webkit.org/show_bug.cgi?id=153404#c9">Comment # 9</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - REGRESSION(r188659): Non main frame scrollable areas don't work for pages restored from the page cache"
href="https://bugs.webkit.org/show_bug.cgi?id=153404">bug 153404</a>
from <span class="vcard"><a class="email" href="mailto:bfulgham@webkit.org" title="Brent Fulgham <bfulgham@webkit.org>"> <span class="fn">Brent Fulgham</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=153404#c8">comment #8</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=271347&action=diff" name="attach_271347" title="Different WIP approach">attachment 271347</a> <a href="attachment.cgi?id=271347&action=edit" title="Different WIP approach">[details]</a></span>
> Different WIP approach
>
> Seems reasonable, but you should make sure that you can actually get back to
> the FrameView in these two destructors (might be too late).</span >
Disconnecting destroyed renderers from the FrameView seems like a good thing to be doing. So if this changes fixes your problem, without reintroducing the issues from <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Scrollable area container is not properly cleared when page is going into the PageCache"
href="show_bug.cgi?id=148182">Bug 148182</a>, then I'm in favor of it.
However, I don't really like that various Render elements have to do work to keep things in proper balance. E.g., RenderListBox and RenderLayer have cleanup code, now DeprecatedPDFPlugin will have special code. Are there others we are missing?
Should ScrollableAreas know about their containing frames, so they can unregister themselves during destruction, rather than doing so in the individual destructors for ScrollableArea descendants?</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>