Issue metadata
Sign in to add a comment
|
When an iframe is hidden, its FrameView calls dispose(); but the FrameView may be reused later |
||||||||||||||||||||||||
Issue descriptionSee: https://codereview.chromium.org/2129033002/#msg3 If display:none is set on an iframe, that calls: LayoutPart::willBeDestroyed() `HTMLFrameOwnerElement::setWidget(nullptr) `moveWidgetToParentSoon(frameView) `frameView->dispose() However, the HTMLFrameOwnerElement retains a reference to the FrameView in its m_contentFrame member variable, and it reuses that FrameView if it subsequently becomes visible. It's bananas to call FrameView::dispose() and then use the FrameView again. The patch linked above tried to fix this, but it caused test failures.
,
Jul 12 2016
First things first: I am now going to start referring to the project as "Root Laser Scrolling". Root laser scrolling will not fix this, but it will fix a related bug, which is that the frameView's scrollbars -- having been destroyed during the call to dispose() -- are not recreated when the iframe becomes visible again ( crbug.com/623280 ).
,
Jul 13 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 17 2017
,
Jul 18
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 18
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by e...@chromium.org
, Jul 12 2016Labels: -Pri-3 OS-All Pri-2
Status: Available (was: Untriaged)