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()) |