Commit 4bac0dbe authored by tkent@chromium.org's avatar tkent@chromium.org

Oilpan: Replace RefPtrs to Node and its subclasses in core/rendering with Oilpan transition types.

BUG=357163

Review URL: https://codereview.chromium.org/318583002

git-svn-id: svn://svn.chromium.org/blink/trunk@175457 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent ca46e7ac
...@@ -55,7 +55,7 @@ namespace WebCore { ...@@ -55,7 +55,7 @@ namespace WebCore {
#ifdef AUTOSIZING_DOM_DEBUG_INFO #ifdef AUTOSIZING_DOM_DEBUG_INFO
class WriteDebugInfoTask : public ExecutionContextTask { class WriteDebugInfoTask : public ExecutionContextTask {
public: public:
WriteDebugInfoTask(PassRefPtr<Element> element, AtomicString value) WriteDebugInfoTask(PassRefPtrWillBeRawPtr<Element> element, AtomicString value)
: m_element(element) : m_element(element)
, m_value(value) , m_value(value)
{ {
...@@ -67,7 +67,7 @@ public: ...@@ -67,7 +67,7 @@ public:
} }
private: private:
RefPtr<Element> m_element; RefPtrWillBePersistent<Element> m_element;
AtomicString m_value; AtomicString m_value;
}; };
......
...@@ -74,7 +74,7 @@ private: ...@@ -74,7 +74,7 @@ private:
LayoutRect m_dirtySourceRect; LayoutRect m_dirtySourceRect;
static RenderLayerFilterInfoMap* s_filterMap; static RenderLayerFilterInfoMap* s_filterMap;
Vector<RefPtr<Element> > m_internalSVGReferences; WillBePersistentHeapVector<RefPtrWillBeMember<Element> > m_internalSVGReferences;
Vector<ResourcePtr<DocumentResource> > m_externalSVGReferences; Vector<ResourcePtr<DocumentResource> > m_externalSVGReferences;
}; };
......
...@@ -81,7 +81,7 @@ private: ...@@ -81,7 +81,7 @@ private:
// so we keep a reference to the Node which caused this custom scrollbar creation. // so we keep a reference to the Node which caused this custom scrollbar creation.
// This will not create a reference cycle as the Widget tree is owned by our containing // This will not create a reference cycle as the Widget tree is owned by our containing
// FrameView which this Node pointer can in no way keep alive. See webkit bug 80610. // FrameView which this Node pointer can in no way keep alive. See webkit bug 80610.
RefPtr<Node> m_owner; RefPtrWillBePersistent<Node> m_owner;
LocalFrame* m_owningFrame; LocalFrame* m_owningFrame;
HashMap<unsigned, RenderScrollbarPart*> m_parts; HashMap<unsigned, RenderScrollbarPart*> m_parts;
......
...@@ -750,7 +750,7 @@ static void writeCounterValuesFromChildren(TextStream& stream, RenderObject* par ...@@ -750,7 +750,7 @@ static void writeCounterValuesFromChildren(TextStream& stream, RenderObject* par
String counterValueForElement(Element* element) String counterValueForElement(Element* element)
{ {
// Make sure the element is not freed during the layout. // Make sure the element is not freed during the layout.
RefPtr<Element> elementRef(element); RefPtrWillBeRawPtr<Element> protector(element);
element->document().updateLayout(); element->document().updateLayout();
TextStream stream; TextStream stream;
bool isFirstCounter = true; bool isFirstCounter = true;
...@@ -765,7 +765,7 @@ String counterValueForElement(Element* element) ...@@ -765,7 +765,7 @@ String counterValueForElement(Element* element)
String markerTextForListItem(Element* element) String markerTextForListItem(Element* element)
{ {
// Make sure the element is not freed during the layout. // Make sure the element is not freed during the layout.
RefPtr<Element> elementRef(element); RefPtrWillBeRawPtr<Element> protector(element);
element->document().updateLayout(); element->document().updateLayout();
RenderObject* renderer = element->renderer(); RenderObject* renderer = element->renderer();
......
...@@ -110,7 +110,7 @@ bool RenderWidget::setWidgetGeometry(const LayoutRect& frame) ...@@ -110,7 +110,7 @@ bool RenderWidget::setWidgetGeometry(const LayoutRect& frame)
return false; return false;
RefPtr<RenderWidget> protector(this); RefPtr<RenderWidget> protector(this);
RefPtr<Node> protectedNode(node()); RefPtrWillBeRawPtr<Node> protectedNode(node());
widget->setFrameRect(newFrame); widget->setFrameRect(newFrame);
return widget->frameRect().size() != newFrame.size(); return widget->frameRect().size() != newFrame.size();
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment