| Index: Source/web/WebLocalFrameImpl.cpp
|
| diff --git a/Source/web/WebLocalFrameImpl.cpp b/Source/web/WebLocalFrameImpl.cpp
|
| index a4ef3ca69934b7c524ef56777bdbae44b3460cab..8525b49f40183e0b4a03932220f8c35b346d605e 100644
|
| --- a/Source/web/WebLocalFrameImpl.cpp
|
| +++ b/Source/web/WebLocalFrameImpl.cpp
|
| @@ -498,7 +498,15 @@ void WebLocalFrameImpl::close()
|
| {
|
| m_client = 0;
|
|
|
| + if (m_frame)
|
| + m_frame->setHasBeenClosed();
|
| +
|
| +#if ENABLE(OILPAN)
|
| + if (m_frame && m_frame->isMainFrame() && viewImpl())
|
| + viewImpl()->clearMainFrame(this);
|
| +#else
|
| deref(); // Balances ref() acquired in WebFrame::create
|
| +#endif
|
| }
|
|
|
| WebString WebLocalFrameImpl::uniqueName() const
|
| @@ -1520,7 +1528,12 @@ WebLocalFrame* WebLocalFrame::create(WebFrameClient* client)
|
|
|
| WebLocalFrameImpl* WebLocalFrameImpl::create(WebFrameClient* client)
|
| {
|
| - return adoptRef(new WebLocalFrameImpl(client)).leakRef();
|
| + WebLocalFrameImpl* frame = new WebLocalFrameImpl(client);
|
| +#if ENABLE(OILPAN)
|
| + return frame;
|
| +#else
|
| + return adoptRef(frame).leakRef();
|
| +#endif
|
| }
|
|
|
| WebLocalFrameImpl::WebLocalFrameImpl(WebFrameClient* client)
|
| @@ -1543,7 +1556,18 @@ WebLocalFrameImpl::~WebLocalFrameImpl()
|
| cancelPendingScopingEffort();
|
| }
|
|
|
| -void WebLocalFrameImpl::setCoreFrame(PassRefPtr<LocalFrame> frame)
|
| +void WebLocalFrameImpl::trace(Visitor* visitor)
|
| +{
|
| +#if ENABLE(OILPAN)
|
| + visitor->trace(m_frame);
|
| + visitor->trace(m_printContext);
|
| + visitor->trace(m_geolocationClientProxy);
|
| +
|
| + WebFrame::traceChildren(visitor, this);
|
| +#endif
|
| +}
|
| +
|
| +void WebLocalFrameImpl::setCoreFrame(PassRefPtrWillBeRawPtr<LocalFrame> frame)
|
| {
|
| m_frame = frame;
|
|
|
| @@ -1566,9 +1590,9 @@ void WebLocalFrameImpl::setCoreFrame(PassRefPtr<LocalFrame> frame)
|
| }
|
| }
|
|
|
| -PassRefPtr<LocalFrame> WebLocalFrameImpl::initializeCoreFrame(FrameHost* host, FrameOwner* owner, const AtomicString& name, const AtomicString& fallbackName)
|
| +PassRefPtrWillBeRawPtr<LocalFrame> WebLocalFrameImpl::initializeCoreFrame(FrameHost* host, FrameOwner* owner, const AtomicString& name, const AtomicString& fallbackName)
|
| {
|
| - RefPtr<LocalFrame> frame = LocalFrame::create(&m_frameLoaderClientImpl, host, owner);
|
| + RefPtrWillBeRawPtr<LocalFrame> frame = LocalFrame::create(&m_frameLoaderClientImpl, host, owner);
|
| setCoreFrame(frame);
|
| frame->tree().setName(name, fallbackName);
|
| // We must call init() after m_frame is assigned because it is referenced
|
| @@ -1578,7 +1602,7 @@ PassRefPtr<LocalFrame> WebLocalFrameImpl::initializeCoreFrame(FrameHost* host, F
|
| return frame;
|
| }
|
|
|
| -PassRefPtr<LocalFrame> WebLocalFrameImpl::createChildFrame(const FrameLoadRequest& request, HTMLFrameOwnerElement* ownerElement)
|
| +PassRefPtrWillBeRawPtr<LocalFrame> WebLocalFrameImpl::createChildFrame(const FrameLoadRequest& request, HTMLFrameOwnerElement* ownerElement)
|
| {
|
| ASSERT(m_client);
|
| WebLocalFrameImpl* webframeChild = toWebLocalFrameImpl(m_client->createChildFrame(this, request.frameName()));
|
| @@ -1589,7 +1613,7 @@ PassRefPtr<LocalFrame> WebLocalFrameImpl::createChildFrame(const FrameLoadReques
|
| // solution. subResourceAttributeName returns just one attribute name. The
|
| // element might not have the attribute, and there might be other attributes
|
| // which can identify the element.
|
| - RefPtr<LocalFrame> child = webframeChild->initializeCoreFrame(frame()->host(), ownerElement, request.frameName(), ownerElement->getAttribute(ownerElement->subResourceAttributeName()));
|
| + RefPtrWillBeRawPtr<LocalFrame> child = webframeChild->initializeCoreFrame(frame()->host(), ownerElement, request.frameName(), ownerElement->getAttribute(ownerElement->subResourceAttributeName()));
|
| // Initializing the core frame may cause the new child to be detached, since
|
| // it may dispatch a load event in the parent.
|
| if (!child->tree().parent())
|
|
|