| Index: third_party/WebKit/Source/core/frame/FrameView.cpp
|
| diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
|
| index 196f562691e386f9bdd7377e29038afb9aec1869..2488510b789f6bdc688144789f9f57de8acbc802 100644
|
| --- a/third_party/WebKit/Source/core/frame/FrameView.cpp
|
| +++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
|
| @@ -350,8 +350,11 @@ void FrameView::dispose() {
|
|
|
| // FIXME: Do we need to do something here for OOPI?
|
| HTMLFrameOwnerElement* ownerElement = m_frame->deprecatedLocalOwner();
|
| - // TODO(dcheng): It seems buggy that we can have an owner element that
|
| - // points to another Widget.
|
| + // TODO(dcheng): It seems buggy that we can have an owner element that points
|
| + // to another Widget. This can happen when a plugin element loads a frame
|
| + // (widget A of type FrameView) and then loads a plugin (widget B of type
|
| + // WebPluginContainerImpl). In this case, the frame's view is A and the frame
|
| + // element's owned widget is B. See https://crbug.com/673170 for an example.
|
| if (ownerElement && ownerElement->ownedWidget() == this)
|
| ownerElement->setWidget(nullptr);
|
|
|
|
|